From: Junio C Hamano <gitster@pobox.com>
To: Jordan DE GEA <jordan.de-gea@grenoble-inp.org>
Cc: mhagger@alum.mit.edu, philipoakley@iee.org, git@vger.kernel.org,
erwan.mathoniere@grenoble-inp.org, samuel.groot@grenoble-inp.org,
tom.russello@grenoble-inp.org, Matthieu.Moy@grenoble-inp.fr
Subject: Re: [PATCHv3] Documentation: triangular workflow
Date: Tue, 07 Jun 2016 12:12:38 -0700 [thread overview]
Message-ID: <xmqqr3c8q0d5.fsf@gitster.mtv.corp.google.com> (raw)
In-Reply-To: <1465288693-6295-1-git-send-email-jordan.de-gea@grenoble-inp.org> (Jordan DE's message of "Tue, 7 Jun 2016 10:38:13 +0200")
Jordan DE GEA <jordan.de-gea@grenoble-inp.org> writes:
> diff --git a/Documentation/gitworkflows.txt b/Documentation/gitworkflows.txt
> index f16c414..3b5fd09 100644
> --- a/Documentation/gitworkflows.txt
> +++ b/Documentation/gitworkflows.txt
> @@ -463,6 +463,156 @@ if you get conflicts: `git am -3` will use index information contained
> in patches to figure out the merge base. See linkgit:git-am[1] for
> other options.
>
> +TRIANGULAR WORKFLOW
> +-------------------
> +
> +In some projects, you cannot push directly to the project but have to
> +suggest your commits to the maintainer (e.g. pull requests).
> +For these projects, it's common to use what's called a *triangular
> +workflow*:
> +
> +- Taking the last version of the project by fetching (e.g.
> + **UPSTREAM**)
"by fetching (e.g. UPSTREAM)" does not finish the sentence nicely.
"... by fetching from **UPSTREAM**" would work better. So would
"Fetching the latest version from the project (e.g. UPSTREAM)".
> +- Writing modifications and push them to a fork (e.g. **PUBLISH**)
> +- Opening a pull request
> +- Checking of changes by the maintainer and, merging them into the
> + **UPSTREAM** repository if accepted
You'd want to end these sentences with full-stop, by the way
> +........................................
> +------------------ -----------------
> +| UPSTREAM | maintainer | PUBLISH |
> +| git/git |- - - - - - - -| me/remote |
> +------------------ ← -----------------
> + \ /
> + \ /
> + fetch↓\ /↑push
> + \ /
> + \ /
> + -------------
> + | LOCAL |
> + -------------
> +........................................
> +
> +Git options to use:
> +~~~~~~~~~~~~~~~~~~~
> + - `branch.<branch>.remote`
> + - `branch.<branch>.pushRemote`
> + - `remote.pushDefault`
> + - `push.default`
> +
> +See linkgit:git-config[1].
The title says "options" but listed are configuration variables and
the referred document is also about git-config. Perhaps retitle it to
Useful configuration variables
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
or something like that?
One thing after reading the above lines that immediately came to my
mind was this:
After listing these four and telling the reader to "See ...", is
there anything else the reader needs to learn from below?
It may make the result a lot more useful document if this gives an
impression to the reader as if you are saying (you do not have to
actually say it) "We will guide you how to set up your workflow in
triangular way, and here are the key configuration variables you
will end up using; don't worry about the details of them, we'll
teach you all about them soon in the following paragraphs."
And I found that "See linkgit:git-config[1]" go directly against
that line of narrative.
> +Push behaviour
> +~~~~~~~~~~~~~~
> + ...
> +
> +Case 2: LOCAL is a clone of **UPSTREAM**
> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> +
> +'In this case, the remote named `origin` corresponds to **UPSTREAM**.'
> +
> +Adding **PUBLISH** remote:
> +
> +===================================
> +* `git remote add publish <PUBLISH_url>`
> +===================================
It may perhaps be only me, but these blind instructions puts me off,
and what I find lacking is "Why should I do this?" "What benefit do
I get by doing this". Start it perhaps like this?
Because you will be pushing into the publish repository
often, instead of having to type its URL every time, you
want a short name you can use to call it.
and then give that "remote add".
> +
> +**Method 1: One option for all branches**
> +
> +Setting `remote.pushDefault` in order to push to **PUBLISH** without
> +argument for push.
> +
> +===================================
> +* `git config remote.pushDefault publish`
> +===================================
This is not too bad, but I'd say
With the "remote add" above, you can say "git push publish"
to push there, instead of saying its URL. But you may want
to be even lazier and say just "git push". To do so:
As a document that is geared toward being a tutorial, I personally
think it is better to stick to one arrangement rather than
presenting case 1/2 as two equivalently valid arrangements and
describe them to equal degree of detail. Otherwise, after finishing
reading Case 1 and immediately reading Case 2 heading, the reader
would start wondering "Which one should I pick? What are the pros
and cons?".
A typical reader of this document would have an upstream in mind,
perhaps a clone of it locally, and may or may not yet have a publish
repository, so one valid choice could be to use Case 2.
Whichever one you choose, the description should not begin with
"pushing". A reader who is the target of this document (i.e. who
owns the LOCAL and PUBLISH repository) begins by cloning and/or
fetching, followed by working on her own change while staying up to
date, and pushing is the last thing she does in the flow.
So I'd recommend reordering the description to
* Introduction. As a summary, here are the four configuration
variables you'll be using to make it easier to arrange.
* "Preparation". Clone from the upstream, create an empty
publish repository and set it as a secondary remote, with
pushdefault pointing at it.
* "Staying up-to-date". You do not have to describe "git fetch"
or "git pull" from the upstream aka origin with too much
detail, as having or not having a publish repository does not
change anything on this side.
* "Making your work available". You would want to reiterate the
fact that "git push" does not go to the upstream but to your
publishing place thanks to the earlier pushdefault
configuration.
* "Alternatively...". In this section, you could mention
possible other arrangements. One could be to set pushdefault
for each and every branch (aka your Case 2/Method 2), which
shouldn't be necesssary because at the beginning of the
document we made it clear that we assume that the reader
cannot push to upstream--the normal place she would be pushing
is to her own publishing place, and configuring "usually all
of them go to my publishing place, but this one alone will go
someplace else" (1) is an advanced workflow element, and more
importantly (2) is not specific to triangular workflow.
Another altenative arrangement worth mentioning may be your
Case 1, i.e. to point at your publish place and a secondary
"upstream" pointing at where your upstream publishes their
work. You can describe what needs to be changed compared to
the above three sections.
so that a first-time reader can learn _one_ workable organization
quickly without having to choose blindly between many choices.
Thanks.
next prev parent reply other threads:[~2016-06-07 19:12 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-26 10:06 [RFC] Triangular Workflow: user friendly full implementation Jordan DE GEA
2016-05-26 11:04 ` Matthieu Moy
2016-05-26 18:30 ` Junio C Hamano
2016-05-30 8:46 ` [RFC] Triangular Workflow UI improvement Jordan DE GEA
2016-05-27 7:32 ` [RFC] Triangular Workflow: user friendly full implementation Philip Oakley
2016-05-30 9:07 ` [RFC] Triangular Workflow UI improvments Jordan DE GEA
2016-05-31 12:28 ` [RFC/PATCH] Triangular Workflow UI improvement: Documentation Jordan DE GEA
2016-05-31 14:33 ` Matthieu Moy
2016-06-01 9:32 ` Jordan DE GEA
2016-06-02 12:02 ` Michael Haggerty
2016-06-03 7:25 ` Philip Oakley
2016-06-03 9:52 ` Jordan DE GEA
2016-06-03 11:36 ` Matthieu Moy
2016-06-03 11:53 ` Jordan DE GEA
2016-06-05 21:28 ` Jordan DE GEA
2016-06-06 7:58 ` Matthieu Moy
2016-06-06 16:46 ` Philip Oakley
2016-06-06 16:54 ` Matthieu Moy
2016-06-06 19:21 ` Philip Oakley
2016-06-07 7:03 ` Matthieu Moy
2016-06-07 20:08 ` Philip Oakley
2016-06-03 15:46 ` Junio C Hamano
2016-06-03 22:16 ` Philip Oakley
2016-06-06 9:48 ` [RFC/PATCHv2] Documentation: triangular workflow Jordan DE GEA
2016-06-06 19:23 ` Junio C Hamano
2016-06-06 22:21 ` Philip Oakley
2016-06-07 6:58 ` Matthieu Moy
2016-06-07 8:02 ` Jordan DE GEA
2016-06-07 8:38 ` [PATCHv3] " Jordan DE GEA
2016-06-07 19:12 ` Junio C Hamano [this message]
2016-06-08 8:37 ` Jordan DE GEA
2016-06-08 13:20 ` Matthieu Moy
2016-06-09 12:35 ` [PATCHv4] " Jordan DE GEA
2016-06-09 17:02 ` Junio C Hamano
2016-06-11 15:58 ` Ramkumar Ramachandra
2016-06-11 19:31 ` Philip Oakley
2016-06-09 18:19 ` Philip Oakley
2016-06-10 16:47 ` Junio C Hamano
2016-06-11 19:25 ` Philip Oakley
2016-06-13 18:35 ` Junio C Hamano
2016-05-30 8:39 ` [RFC] Triangular Workflow UI improvement Jordan DE GEA
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=xmqqr3c8q0d5.fsf@gitster.mtv.corp.google.com \
--to=gitster@pobox.com \
--cc=Matthieu.Moy@grenoble-inp.fr \
--cc=erwan.mathoniere@grenoble-inp.org \
--cc=git@vger.kernel.org \
--cc=jordan.de-gea@grenoble-inp.org \
--cc=mhagger@alum.mit.edu \
--cc=philipoakley@iee.org \
--cc=samuel.groot@grenoble-inp.org \
--cc=tom.russello@grenoble-inp.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).