( )โš™ D10161 push: allow to specify multiple destinations

This is an archive of the discontinued Mercurial Phabricator instance.

push: allow to specify multiple destinations
ClosedPublic

Authored by marmoute on Mar 11 2021, 11:52 AM.

Details

Summary

I end up needing that on a regular basis and it turn out to be very simple to
implement. See documentation and test for details.

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.Mar 11 2021, 11:52 AM
danchr added a subscriber: danchr.Mar 11 2021, 12:12 PM
danchr added inline comments.
mercurial/commands.py
5666

DESTINATIONS should be plural; s/on /one/

Also, this seems a bit verbose. I'd suggest:

When passed multiple destinations, push will process them one after the other, but stop should an error occur.

baymax updated this revision to Diff 26252.Mar 11 2021, 12:45 PM

โœ… refresh by Heptapod after a successful CI run (๐Ÿ™ ๐Ÿ’š)

Good idea, this is updated.

Alphare accepted this revision.Mar 16 2021, 5:52 AM
Alphare added a subscriber: Alphare.

I've been wanting this feature for a while, thanks!

baymax updated this revision to Diff 26485.Mar 17 2021, 11:01 AM

โœ… refresh by Heptapod after a successful CI run (๐Ÿ™ ๐Ÿ’š)

This revision was not accepted when it landed; it landed in state Needs Review.
This revision was automatically updated to reflect the committed changes.