From: Thomas Rast <trast@student.ethz.ch>
To: <rocketraman@fastmail.fm>
Cc: <git@vger.kernel.org>
Subject: Re: [PATCHv2] Update gitworkflows man page to include release workflow
Date: Wed, 11 Nov 2009 21:41:58 +0100 [thread overview]
Message-ID: <200911112142.00209.trast@student.ethz.ch> (raw)
In-Reply-To: <1257869339-15999-2-git-send-email-rocketraman@fastmail.fm>
It's nice to see someone work on this manpage :-) I sadly do not have
the time to read the whole patch right now, though I'll try and catch
up tomorrow or so. In the meantime I do have one remark:
rocketraman@fastmail.fm wrote:
> +The maintainer may use the following release workflow:
> +
> +He first tags the tip of 'master' with a release tag, then he updates
> +the 'maint' branch to the current tip of 'master' for managing future
> +maintenance fixes on the current release, and lastly he optionally
> +rebuilds 'next' from the tip of 'master'.
The current gitworkflows is mostly formulated in the imperative, as in
To test the interaction of several topics, merge them into a
throw-away branch. You must never base any work on such a branch!
or by directly describing the tools in the third person, as in
* linkgit:git-push[1] copies your branches to a remote repository,
usually to one that can be read by all involved parties;
It would certainly be nice to be somewhat consistent. Since at first
glance your description is aimed at the maintainer himself, I assume
that would mostly mean addressing the maintainer as "you", and
formulating the rules in the imperative.
--
Thomas Rast
trast@{inf,student}.ethz.ch
next prev parent reply other threads:[~2009-11-11 20:42 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-10 16:08 Update gitworkflows man page to include release workflow rocketraman
2009-11-10 16:08 ` [PATCHv2] " rocketraman
2009-11-10 18:06 ` Štěpán Němec
2009-11-10 18:10 ` Raman Gupta
2009-11-11 13:05 ` Thiago Farina
2009-11-11 15:23 ` Raman Gupta
2009-11-11 19:43 ` Junio C Hamano
2009-11-12 0:32 ` Raman Gupta
2009-11-12 8:03 ` Junio C Hamano
2009-11-12 8:10 ` Thomas Rast
2009-11-12 16:54 ` Raman Gupta
2009-11-11 20:41 ` Thomas Rast [this message]
2009-11-12 8:27 ` Thomas Rast
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200911112142.00209.trast@student.ethz.ch \
--to=trast@student.ethz.ch \
--cc=git@vger.kernel.org \
--cc=rocketraman@fastmail.fm \
/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.
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 NNTP newsgroup(s).