Page MenuHomePhabricator

infinitepush: use the new function to determine push destination
ClosedPublic

Authored by marmoute on Apr 13 2021, 8:19 AM.

Details

Summary

Since 066b8d8f75b8, the push command accept multiple destination. infinitepush
was not aware of that. We now use the new urlutil.get_push_paths function to
determine the push destination, fixing the issue. This will also make future
evolution of that logic transparent for infinitepush

We still disallow push to multiple destinations if infinite push is enabled
because I don't know what this means for infinite push. However user will now
get a clear error message instead of a crash.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

marmoute created this revision.Apr 13 2021, 8:19 AM
pulkit accepted this revision.Apr 13 2021, 8:52 AM
This revision is now accepted and ready to land.Apr 13 2021, 8:52 AM