[PATCH V4] forget: add --dry-run mode
Anton Shestakov
av6 at dwimlabs.net
Tue Mar 13 08:50:52 EDT 2018
On Tue, 13 Mar 2018 15:43:02 +0800
Anton Shestakov <av6 at dwimlabs.net> wrote:
> > + rejected = cmdutil.forget(ui, repo, m, prefix="", explicitonly=False, dryrun=dryrun)[0]
>
> This line is too long, don't forget to also run test-check-commit.t.
I saw this already fixed, but I'd like to get this right.
test-check-code.t is the test that would detect lines of code being too
long (not *-commit.t), but running all of the test-check-*.t before
sending patches is a good idea. Small issues can be fixed when
reviewers queue patches, but not all of check-code.py (as an example)
is style checks; some of the rules are important for compatibility
(Python 2 vs 3, and running test suite on different OSes), and they are
put there because they can be tricky to spot during manual review.
More information about the Mercurial-devel
mailing list