Re: [PATCH 1/2] push: avoid suggesting "merging" remote changes
- From
Matthieu Moy <matthieu.moy@grenoble-inp.fr>
- Date
- Jul 8, 2013, 13:47 UTC
- Message-ID
- <vpqd2qtkvtk.fsf@anie.imag.fr>
- In-Reply-To
- <2a6f4a3c31e667aa03b06ce63ae5319a25beaca0.1373223663.git.john@keeping.me.uk>
John Keeping <john@keeping.me.uk> writes:
Show 6 quoted lines
> static const char message_advice_pull_before_push[] =
> N_("Updates were rejected because the tip of your current branch is behind\n"
> - "its remote counterpart. Merge the remote changes (e.g. 'git pull')\n"
> - "before pushing again.\n"
> + "its remote counterpart. Integrate the remote changes (e.g.\n"
> + "'git pull ...') before pushing again.\n"To me, "merge" includes "rebase", so I'd say the merge -> integrate change is not needed, but I have nothing against it either.
The "..." added are a bit weird with the quotes around. Quotes may suggest that the content is to be taken literally, which is not the case anymore. Not a real objection anyway, just thinking aloud.
-- Matthieu Moy http://www-verimag.imag.fr/~moy/