* [PATCH] docs: Clarify "preserve" option wording for git-pull
@ 2015-03-26 11:04 Sebastian Schuberth
2015-03-26 17:37 ` Junio C Hamano
0 siblings, 1 reply; 4+ messages in thread
From: Sebastian Schuberth @ 2015-03-26 11:04 UTC (permalink / raw)
To: Git Mailing List; +Cc: Junio C Hamano
Before this path, the "also" sounded as if "preserve" was doing a rebase
as additional work that "true" would not do. Clarify this by saying
"still" instead of "also".
Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
---
Documentation/git-pull.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/git-pull.txt b/Documentation/git-pull.txt
index d3d236c..96829be 100644
--- a/Documentation/git-pull.txt
+++ b/Documentation/git-pull.txt
@@ -111,8 +111,8 @@ include::merge-options.txt[]
was rebased since last fetched, the rebase uses that information
to avoid rebasing non-local changes.
+
-When preserve, also rebase the current branch on top of the upstream
-branch, but pass `--preserve-merges` along to `git rebase` so that
+When preserve, still rebase the current branch on top of the upstream
+branch, but also pass `--preserve-merges` along to `git rebase` so that
locally created merge commits will not be flattened.
+
When false, merge the current branch into the upstream branch.
--
1.9.5.msysgit.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] docs: Clarify "preserve" option wording for git-pull
2015-03-26 11:04 [PATCH] docs: Clarify "preserve" option wording for git-pull Sebastian Schuberth
@ 2015-03-26 17:37 ` Junio C Hamano
2015-03-26 20:11 ` Sebastian Schuberth
0 siblings, 1 reply; 4+ messages in thread
From: Junio C Hamano @ 2015-03-26 17:37 UTC (permalink / raw)
To: Sebastian Schuberth; +Cc: Git Mailing List
Sebastian Schuberth <sschuberth@gmail.com> writes:
> Before this path, the "also" sounded as if "preserve" was doing a rebase
> as additional work that "true" would not do. Clarify this by saying
> "still" instead of "also".
I agree that the original "also" is confusing. I however wonder if
we even want "still", though. For that matter, I doubt if we even
want "also" in front of "pass". "When set to preserve, rebase and
pass the extra flag" sounds as clear as, if not clearer than, "When
set to preserve, still rebase and also pass the extra flag", at
least to me.
Thanks.
> Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
> ---
> Documentation/git-pull.txt | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/git-pull.txt b/Documentation/git-pull.txt
> index d3d236c..96829be 100644
> --- a/Documentation/git-pull.txt
> +++ b/Documentation/git-pull.txt
> @@ -111,8 +111,8 @@ include::merge-options.txt[]
> was rebased since last fetched, the rebase uses that information
> to avoid rebasing non-local changes.
> +
> -When preserve, also rebase the current branch on top of the upstream
> -branch, but pass `--preserve-merges` along to `git rebase` so that
> +When preserve, still rebase the current branch on top of the upstream
> +branch, but also pass `--preserve-merges` along to `git rebase` so that
> locally created merge commits will not be flattened.
> +
> When false, merge the current branch into the upstream branch.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] docs: Clarify "preserve" option wording for git-pull
2015-03-26 17:37 ` Junio C Hamano
@ 2015-03-26 20:11 ` Sebastian Schuberth
2015-03-26 20:22 ` Junio C Hamano
0 siblings, 1 reply; 4+ messages in thread
From: Sebastian Schuberth @ 2015-03-26 20:11 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Git Mailing List
Before this path, the "also" sounded as if "preserve" was doing a rebase
as additional work that "true" would not do. Clarify this by omitting the
"also" and rewording the sentence a bit.
Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
---
Documentation/git-pull.txt | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/Documentation/git-pull.txt b/Documentation/git-pull.txt
index d3d236c..712ab4b 100644
--- a/Documentation/git-pull.txt
+++ b/Documentation/git-pull.txt
@@ -111,9 +111,8 @@ include::merge-options.txt[]
was rebased since last fetched, the rebase uses that information
to avoid rebasing non-local changes.
+
-When preserve, also rebase the current branch on top of the upstream
-branch, but pass `--preserve-merges` along to `git rebase` so that
-locally created merge commits will not be flattened.
+When set to preserve, rebase with the `--preserve-merges` option passed
+to `git rebase` so that locally created merge commits will not be flattened.
+
When false, merge the current branch into the upstream branch.
+
--
1.9.5.msysgit.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] docs: Clarify "preserve" option wording for git-pull
2015-03-26 20:11 ` Sebastian Schuberth
@ 2015-03-26 20:22 ` Junio C Hamano
0 siblings, 0 replies; 4+ messages in thread
From: Junio C Hamano @ 2015-03-26 20:22 UTC (permalink / raw)
To: Sebastian Schuberth; +Cc: Git Mailing List
Thanks; this time I do not see whitespace breakages ;-)
Will queue with a minimum tweak of the log message.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-03-26 20:22 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-26 11:04 [PATCH] docs: Clarify "preserve" option wording for git-pull Sebastian Schuberth
2015-03-26 17:37 ` Junio C Hamano
2015-03-26 20:11 ` Sebastian Schuberth
2015-03-26 20:22 ` Junio C Hamano
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.