Re: [PATCH v2] pull: require choice between rebase/merge on non-fast-forward pull

From: Junio C Hamano <gitster@pobox.com>
Date: Jul 19, 2013, 22:20 UTC
Message-ID: <7vzjtitco6.fsf@alter.siamese.dyndns.org>
In-Reply-To: <CAPig+cQEtKc+tfDgqVWYL2JtxXc=wvS=P7_O=XJzizz1BN=n4A@mail.gmail.com>

Eric Sunshine <sunshine@sunshineco.com> writes:

Show 8 quoted lines

> Dropping the parenthetical comment might improve flow slightly:
>
>     Without repository or branch on the command line, `git pull`
>     needs to be told how to integrate the changes with your history,
>     via either `--merge` or `--rebase`.
>
> With or without mention of the configuration options, either phrasing
> seems pretty easy to digest.

Yeah, that reads much better, but I do prefer to see something that explains this is often "just make sure you use the one that suits your project and always use that". How about something like this?

    With no repository or branch on the command line, `git pull` needs
    to be told how to integrate the changes with your history.

    This can be done via either `--merge` or `--rebase` option, but most
    people would want to decide which method matches the workflow of the
    project once, and set the configuration variable `pull.rebase` or
    `branch.<name>.rebase` to stick to it; see linkgit:git-config[1].