Re: [PATCH] push: improve consistency of output when "up to date"
- From
Taylor Blau <me@ttaylorr.com>
- Date
- Jan 11, 2024, 21:43 UTC
- Message-ID
- <ZaBhHSCT7EOgTo/N@nand.local>
- In-Reply-To
- <pull.1638.git.1705008449995.gitgitgadget@gmail.com>
On Thu, Jan 11, 2024 at 09:27:29PM +0000, Benji Kay via GitGitGadget wrote:
Show 28 quoted lines
> From: okaybenji <okaybenji@gmail.com>
>
> When one issues the pull command, one may see "Already up to date."
> When issuing the push command, one may have seen "Everything up-to-date".
> To improve consistency, "Everything up to date." is printed instead.
> (The hyphens have been removed, and a period has been added.)
>
> Signed-off-by: okaybenji <okaybenji@gmail.com>
> ---
> push: improve consistency of output when "up to date"
>
> Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1638%2Fokaybenji%2Fup-to-date-v1
> Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1638/okaybenji/up-to-date-v1
> Pull-Request: https://github.com/gitgitgadget/git/pull/1638
>
> transport.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/transport.c b/transport.c
> index bd7899e9bf5..c42cb4e58b4 100644
> --- a/transport.c
> +++ b/transport.c
> @@ -1467,7 +1467,7 @@ int transport_push(struct repository *r,
> if (porcelain && !push_ret)
> puts("Done");
> else if (!quiet && !ret && !transport_refs_pushed(remote_refs))
> - fprintf(stderr, "Everything up-to-date\n");
> + fprintf(stderr, "Everything up to date.\n");Between the two, I have a vague preference towards "up-to-date", which would suggest changing the pull command's output to read "Already up-to-date". Personally I think that neither of them should include a period in their output, but whichever we decide should be done so consistently between the two.
Also, should this string be marked for translation?
Thanks, Taylor