zsh-workers
 help / color / mirror / code / Atom feed
* git again
@ 2013-02-12 11:01 Oliver Kiddle
  2013-02-12 20:39 ` Phil Pennock
  0 siblings, 1 reply; 2+ messages in thread
From: Oliver Kiddle @ 2013-02-12 11:01 UTC (permalink / raw)
  To: Zsh workers

I've cleaned up the duplication in the zsh-development-guide regarding
git usage and workflows. I'm not posting a patch because I think it'd be
better to discuss the content as it now stands than to pick over the
changes (That is, if further discussion is necessary which I can't say
I'm enthused by).

As confirmed with Peter, the intention is to keep things as unchanged as
possible from CVS for now. This means manual ChangeLog updates (though
not as a separate commit).

With this in mind, I've simplified the git branch usage written by Phil
to recommend git merge --squash. Note to git newcomers, this is still a
useful approach to follow: I use local branches even for fairly trivial
changes and check in regularly at intermediate states; often as a way to
push changes between Linux and Solaris.

A "Sharing Feature Branches" section can be added later once we start doing
that for non-trivial changes.

Anyway, I'll leave this for a week or so and then we can look at making
the actual switch.

Oliver


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: git again
  2013-02-12 11:01 git again Oliver Kiddle
@ 2013-02-12 20:39 ` Phil Pennock
  0 siblings, 0 replies; 2+ messages in thread
From: Phil Pennock @ 2013-02-12 20:39 UTC (permalink / raw)
  To: Oliver Kiddle; +Cc: Zsh workers

On 2013-02-12 at 12:01 +0100, Oliver Kiddle wrote:
> With this in mind, I've simplified the git branch usage written by Phil
> to recommend git merge --squash. Note to git newcomers, this is still a
> useful approach to follow: I use local branches even for fairly trivial
> changes and check in regularly at intermediate states; often as a way to
> push changes between Linux and Solaris.

LGTM, +1, Endorsed, whatever-the-current-vogue-is.  I like the new text.

(I suspect we'll end up suggesting that if you don't use "merge
 --squash" then you might want "rebase -i master" instead of a plain
 rebase, to be able to clean up the history with a tool that's both
 powerful and simple for that amount of power.  But it doesn't need to
 be in the doc for now: endorsed as-is.)

-Phil


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-02-12 20:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-02-12 11:01 git again Oliver Kiddle
2013-02-12 20:39 ` Phil Pennock

Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

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).