[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#57400: 29.0.50; Support sending patches from VC directly
From: |
Eli Zaretskii |
Subject: |
bug#57400: 29.0.50; Support sending patches from VC directly |
Date: |
Thu, 13 Oct 2022 23:33:34 +0300 |
> Cc: Robert Pluim <rpluim@gmail.com>, 57400@debbugs.gnu.org,
> Antoine Kalmbach <ane@iki.fi>
> From: Philip Kaludercic <philipk@posteo.net>
> Date: Thu, 13 Oct 2022 20:25:03 +0000
>
> -(defun vc-git-symbolic-commit (commit)
> +(defun vc-git-symbolic-commit (commit &optional force)
> "Translate COMMIT string into symbolic form.
> -Returns nil if not possible."
> +Returns nil if not possible.
And if possible? I guess the first sentence lacks something it should
say about that? (Also, we use "Return", not "Returns", to be
consistent with "Translate".)
> If the optional argument FORCE is
> +non-nil, revisions containing positional
> +arguments (e.g. \"master~8\") will also be accepted."
It is not a good idea to use "also" when you didn't explain before
that what this does without FORCE. (Maybe this should be part of the
explanation of what you mean by "not possible" above?)
Also, there's a passive tense here...
And finally, you use "positional arguments", but "man gitrevisions"
doesn't use this term at all. So maybe we should use a more accepted
terminology, or at least provide more than just one example to explain
that via the examples?
Thanks.
- bug#57400: 29.0.50; Support sending patches from VC directly, (continued)
- bug#57400: 29.0.50; Support sending patches from VC directly, Philip Kaludercic, 2022/10/11
- bug#57400: 29.0.50; Support sending patches from VC directly, Juri Linkov, 2022/10/11
- bug#57400: 29.0.50; Support sending patches from VC directly, Philip Kaludercic, 2022/10/11
- bug#57400: 29.0.50; Support sending patches from VC directly, Richard Stallman, 2022/10/12
- bug#57400: 29.0.50; Support sending patches from VC directly, Juri Linkov, 2022/10/13
- bug#57400: 29.0.50; Support sending patches from VC directly, Richard Stallman, 2022/10/13
- bug#57400: 29.0.50; Support sending patches from VC directly, Philip Kaludercic, 2022/10/13
- bug#57400: 29.0.50; Support sending patches from VC directly, Juri Linkov, 2022/10/13
- bug#57400: 29.0.50; Support sending patches from VC directly, Philip Kaludercic, 2022/10/13
- bug#57400: 29.0.50; Support sending patches from VC directly, Philip Kaludercic, 2022/10/13
- bug#57400: 29.0.50; Support sending patches from VC directly,
Eli Zaretskii <=
- bug#57400: 29.0.50; Support sending patches from VC directly, Philip Kaludercic, 2022/10/13
- bug#57400: 29.0.50; Support sending patches from VC directly, Eli Zaretskii, 2022/10/14
- bug#57400: 29.0.50; Support sending patches from VC directly, Richard Stallman, 2022/10/14
- bug#57400: 29.0.50; Support sending patches from VC directly, Philip Kaludercic, 2022/10/14
- bug#57400: 29.0.50; Support sending patches from VC directly, Eli Zaretskii, 2022/10/15
- bug#57400: 29.0.50; Support sending patches from VC directly, Philip Kaludercic, 2022/10/15
- bug#57400: 29.0.50; Support sending patches from VC directly, Eli Zaretskii, 2022/10/15
- bug#57400: 29.0.50; Support sending patches from VC directly, Philip Kaludercic, 2022/10/15
- bug#57400: 29.0.50; Support sending patches from VC directly, Eli Zaretskii, 2022/10/15
- bug#57400: 29.0.50; Support sending patches from VC directly, Philip Kaludercic, 2022/10/15