Re: [PATCH v3 4/4] i18n: transfer variables into placeholders in command synopsis
On 30/01/2022 22:01, Jean-Noël Avila via GitGitGadget wrote:
> From: =?UTF-8?q?Jean-No=C3=ABl=20Avila?= <jn.avila@free.fr>
>
> This applies the style guide for documentation.
I don't really understand the title of this patch. Should 'transfer' be
'transform'? All the changes look fine but seem to be two sorts of
changes mixed together
- adding '<>' around arguments where they are missing
- converting plurals followed by '...' into singulars followed by '...'
Best Wishes
Phillip
Show 50 quoted lines
> Signed-off-by: Jean-Noël Avila <jn.avila@free.fr>
> ---
> builtin/fast-export.c | 2 +-
> builtin/reflog.c | 4 ++--
> builtin/rev-list.c | 2 +-
> 3 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/builtin/fast-export.c b/builtin/fast-export.c
> index 9f1c730e587..510139e9b54 100644
> --- a/builtin/fast-export.c
> +++ b/builtin/fast-export.c
> @@ -26,7 +26,7 @@
> #include "commit-slab.h"
>
> static const char *fast_export_usage[] = {
> - N_("git fast-export [rev-list-opts]"),
> + N_("git fast-export [<rev-list-opts>]"),
> NULL
> };
>
> diff --git a/builtin/reflog.c b/builtin/reflog.c
> index ee5ee8d8cf6..343a10d371b 100644
> --- a/builtin/reflog.c
> +++ b/builtin/reflog.c
> @@ -17,10 +17,10 @@ static const char reflog_expire_usage[] =
> N_("git reflog expire [--expire=<time>] "
> "[--expire-unreachable=<time>] "
> "[--rewrite] [--updateref] [--stale-fix] [--dry-run | -n] "
> - "[--verbose] [--all] <refs>...");
> + "[--verbose] [--all] <ref>...");
> static const char reflog_delete_usage[] =
> N_("git reflog delete [--rewrite] [--updateref] "
> - "[--dry-run | -n] [--verbose] <refs>...");
> + "[--dry-run | -n] [--verbose] <ref>...");
> static const char reflog_exists_usage[] =
> N_("git reflog exists <ref>");
>
> diff --git a/builtin/rev-list.c b/builtin/rev-list.c
> index 777558e9b06..38528c7f156 100644
> --- a/builtin/rev-list.c
> +++ b/builtin/rev-list.c
> @@ -20,7 +20,7 @@
> #include "packfile.h"
>
> static const char rev_list_usage[] =
> -"git rev-list [OPTION] <commit-id>... [ -- paths... ]\n"
> +"git rev-list [<options>] <commit-id>... [-- <path>...]\n"
> " limiting output:\n"
> " --max-count=<n>\n"
> " --max-age=<epoch>\n"