This is an archive of the discontinued Mercurial Phabricator instance.

Differential D7637

pathbomb: use cmdutil.check_at_most_one_arg()
ClosedPublic

Authored by martinvonz on Dec 13 2019, 2:53 AM.

Details

Reviewers
pulkit
Group Reviewers
hg-reviewers
Commits
rHG3781da40eaa6: patchbomb: use cmdutil.check_at_most_one_arg()

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

martinvonz created this revision.Dec 13 2019, 2:53 AM
martinvonz retitled this revision from pathbomb: use cmdutil.check_unique_argument() to pathbomb: use cmdutil.check_at_most_one_arg().Dec 16 2019, 6:18 PM
martinvonz updated this revision to Diff 18763.
pulkit accepted this revision.Dec 17 2019, 2:11 AM
This revision is now accepted and ready to land.Dec 17 2019, 2:11 AM
pulkit added a comment.Dec 17 2019, 2:26 AM

Renaming pathbomb to patchbomb in message title.

martinvonz added a commit: rHG3781da40eaa6: patchbomb: use cmdutil.check_at_most_one_arg().Dec 17 2019, 2:58 AM
Closed by commit rHG3781da40eaa6: patchbomb: use cmdutil.check_at_most_one_arg() (authored by martinvonz). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mhgext/patchbomb.py (3 lines)

Diff 18672

hgext/patchbomb.py

if outgoing and bundle: if outgoing and bundle:
raise error.Abort( raise error.Abort(
_( _(
b"--outgoing mode always on with --bundle;" b"--outgoing mode always on with --bundle;"
b" do not re-specify --outgoing" b" do not re-specify --outgoing"
) )
) )
if rev and bookmark: cmdutil.check_unique_argument(opts, b'rev', b'bookmark')
raise error.Abort(_(b"-r and -B are mutually exclusive"))
if outgoing or bundle: if outgoing or bundle:
if len(revs) > 1: if len(revs) > 1:
raise error.Abort(_(b"too many destinations")) raise error.Abort(_(b"too many destinations"))
if revs: if revs:
dest = revs[0] dest = revs[0]
else: else:
dest = None dest = None