[PATCH] update: correct description of --check option

Martin von Zweigbergk martinvonz at google.com
Wed Apr 27 23:56:09 EDT 2016


On Wed, Apr 27, 2016 at 5:31 PM, timeless <timeless at gmail.com> wrote:
> Martin von Zweigbergk wrote:
>> The current "update across branches if no uncommitted changes" makes
>> it sound like updating across branches (with no uncommitted changes)
>
> Commit messages should describe the state with the changes applied as
> present tense.
> This quoted text doesn't do that.

Huh, I had no idea. Here's a rewrite from that perspective for someone
to change to in flight (I left present (?) tense in the sentence that
starts with "Let's" since I've seen that used frequently):

The old "update across branches if no uncommitted changes" made
it sound like updating across branches (with no uncommitted changes)
was allowed only with this option, which was not true. Also, the option
did not care whether it was linear or across branches. Instead, it
checked that there were no uncommitted changes. Let's explain what it
does instead of trying to suggest what happens without it.


More information about the Mercurial-devel mailing list