Thread (4 messages) flat view 4 messages, 4 authors, 2016-06-15
  • (off-list ancestor, not in this archive)
  • Re: git push · Junio C Hamano <hidden> · 2016-06-15
  • Re: git push · Jeff King <hidden> · 2016-06-15
  • Re: git push · John Tapsell <hidden> · 2016-06-15
  • Re: git push · Jay Soffian <hidden> · 2016-06-15

Re: git push

From: Jeff King <hidden>
Date: 2016-06-15 22:46:17

On Tue, Feb 24, 2009 at 11:44:44PM -0800, Junio C Hamano wrote:
quoted
Presumably the obvious is that it might be confusing to existing
users?  Perhaps, but it doesn't cause any damage.  It's moving to a
'safer' default.
No, it is not merely confusing but is outright dangerous to people who
expect the "matching refs" behaviour.  It is not safer at all.
I think this is a very good reason not to change the default "push"
behavior.
And this is not about punishing.  It is about getting into a different
mindset.  Unlike CVS/SVN, committing and publishing can be made into
different phases with git, and not pushing too early allows you produce a
lot better results.  "I want to push only this branch" is often (not
always, but "often" stands with strong correlation) a sign that other
things are not ready, and by definition you couldn't have thought through
interactions between what you _think_ is ready (i.e. the current branch)
and the other ones that are not ready.  In other words, it is about
encouraging people to think things through before publishing.
But I don't buy this at all. It is totally dependent on workflow and how
you use branches. That is, the "readiness" of two branches may be
totally unrelated. One may be a short-term branch for today's work, and
the other may be a long-running branch that you have made a WIP commit
on. You may even have that WIP commit sitting there for days.

When you think about "is my current branch ready to push?" there is no
reason for you to think of that other long-running branch that you
haven't seen for days.

-Peff
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help