D7641: rebase: use cmdutil.check_at_most_one_arg() for --confirm/--dry-run

Yuya Nishihara yuya at tcha.org
Tue Dec 17 10:30:32 EST 2019


> > Spotted in a later patch, need to replace `_` back to `-` before printing output.
> 
> Strings are immutable in Python, so it wasn't actually replaced, right? Or do I misunderstand your concern?

Maybe he meant `opts[b'dry_run']` vs `--dry-run`. `*args` should follow
underscore naming, but the printed option names don't.


More information about the Mercurial-devel mailing list