D10391: outgoing: accept multiple destinations

marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This align the behavior of `hg outgoing` with the one of `hg incoming`. In addition this prepare the introduction of having simple `path` resolve to multiple de

D10390: outgoing: pass subrepo path using function argument instead of abssource hack

marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This is clearer, remove the needs for the `repo._subtoppath` hack and will make our live easier when making `outgoing` accept multiple destinations. We ne

D10388: split: fix issue with empty splits adjusting phases

spectral created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10388 AFFECTED FILES hgext/split.py tests/test-split.t CHANGE DETAILS diff --g

D10389: split: avoid strip if split is a no-op (identical to original)

spectral created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10389 AFFECTED FILES hgext/split.py tests/test-split.t CHANGE DETAILS diff --g

D10387: split: add test demonstrating issue with empty splits adjusting phases

spectral created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10387 AFFECTED FILES tests/test-split.t CHANGE DETAILS diff --git a/tests/test-s

D10386: tests: avoid use of "python", which may not even be installed

spectral created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10386 AFFECTED FILES tests/test-convert-cvs.t tests/test-merge-tools.t CHANGE D

D10385: push-dests: move the code around missing default dest inside `get_push_paths`

marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This function has a clear semantic and moving the code dealing with this inside it will help reduce duplication. REPOSITORY rHG Mercurial BRANCH default

D10383: outgoing: merge the code handling --graph with the main one

marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The --graph code had its own copy of the logic. With the previous reorganisation of the code, we can now merge it with the main code, reducing fragile compli

D10384: push-dests: rework the handling of default value

marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This new core is more straightforward and doing this early will make the next changeset simpler. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D10382: outgoing: move filtering logic in its own function

marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This move code dedicated to a single purpose together and make the main code simpler. Right when we are getting ready to make it more complex :-D REPOSITORY

D10380: outgoing: remove some early return

marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Since 066b8d8f75b8 , the push command accept multiple destination. However `hg out

D10381: outgoing: make `recurse` a real function

marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY If we want to use this in a loop, we need to be able to pass argument. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm

D10378: urlutil: add a `get_pull_paths` to perform the pull destination logic

marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY As is this changeset does not change anything. However having an official empty point will help unifying the logic and encapsulate the details and update the

D10379: infinitepush: use the new function to determine push destination

marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Since 066b8d8f75b8 , the push command accept multiple destination. `infinitepush`

D10377: urlutil: add a `get_push_paths` to perform the push destination logic

marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY As is this changeset does not change anything. However having an official empty point will help unifying the logic and encapsulate the details and update the

Re: development heads up: push/pull improvements

On 4/13/21 3:03 AM, Manuel Jacob wrote: On 17/03/2021 16.05, Pierre-Yves David wrote: Hello everyone, I have been working on improvements to push and pull. The core goal can be summarized as: * make it simpler to push/pull to multiple destinations * make it simpler to control default behavi