Based on the current confusion about branches, I think this problem can be considered "stops good developers from working", and thus qualifies as Critical.
Labels: -Type-Enhancement Type-Critical
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Based on the current confusion about branches, I think this problem can be considered "stops good developers from working", and thus qualifies as Critical.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
based on my time and understanding of git, this is taking me a while to update. If somebody else wants to grab the patch and fix it, that'd be great. Otherwise we're looking at Jan 6 at the earliest (that's when I'm back in Glasgow).
Owner: --- Status: Accepted
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
I think the notion of a "countdown" has played its part for this issue. I suggest that we push it; the only remaining quibbles might be over slight phrasing things that can be edited later if necessary/
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Originally posted by: lilypond...@gmail.com
Patchy the autobot says: LGTM.
Labels: Patch-review
Originally posted by: lilypond...@gmail.com
CG: explanation of branches for the impatient
http://codereview.appspot.com/5484043
Labels: Patch-new
Originally posted by: lilypond...@gmail.com
Patchy the autobot says: fails make
Labels: Patch-needs_work
Originally posted by: lilypond...@gmail.com
CG: explanation of branches for the impatient
http://codereview.appspot.com/5484043
Labels: Patch-new
Originally posted by: lilypond...@gmail.com
Patchy the autobot says: does not apply to master
Labels: Patch-needs_work
Originally posted by: lilypond...@gmail.com
CG: explanation of branches for the impatient
http://codereview.appspot.com/5484043
Labels: Patch-new
Originally posted by: lilypond...@gmail.com
Patchy the autobot says: LGTM. turns out the problem was in the checking script, not the patch itself.
Labels: Patch-review
Originally posted by: ColinPKC...@gmail.com
(No comment was entered for this change.)
Labels: -Patch-review Patch-countdown
Originally posted by: gra...@percival-music.ca
Keith identified some problems in this.
Labels: -Patch-countdown Patch-needs_work
Originally posted by: pkx1...@gmail.com
(No comment was entered for this change.)
Owner: gra...@percival-music.ca
Status: Started
Originally posted by: gra...@percival-music.ca
Based on the current confusion about branches, I think this problem can be considered "stops good developers from working", and thus qualifies as Critical.
Labels: -Type-Enhancement Type-Critical
Originally posted by: gra...@percival-music.ca
Based on the current confusion about branches, I think this problem can be considered "stops good developers from working", and thus qualifies as Critical.
Originally posted by: gra...@percival-music.ca
based on my time and understanding of git, this is taking me a while to update. If somebody else wants to grab the patch and fix it, that'd be great. Otherwise we're looking at Jan 6 at the earliest (that's when I'm back in Glasgow).
Owner: ---
Status: Accepted
Originally posted by: janek.li...@gmail.com
Graham,
by "grab the patch and fix it" do you mean "download a patch, fix it according to your taste and upload it in a new Rietveld issue"?
Originally posted by: gra...@percival-music.ca
The patch from 4 weeks has been uploaded to dev/cg-branches
Originally posted by: Carl.D.S...@gmail.com
Issue 2100: Explanation of branches for CG
Explain branches, by Graham Percival
Changes to git commands by Carl Sorensen
http://codereview.appspot.com/5539062
Labels: Patch-new
Originally posted by: Carl.D.S...@gmail.com
Issue 2100: Explanation of branches for CG
Explain branches, by Graham Percival
Changes to git commands by Carl Sorensen
http://codereview.appspot.com/5539062
Originally posted by: Carl.D.S...@gmail.com
Issue 2100: Explanation of branches for CG
Explain branches, by Graham Percival
Changes to git commands by Carl Sorensen
http://codereview.appspot.com/5539062
Originally posted by: lilypond...@gmail.com
Patchy the autobot says: LGTM.
Labels: Patch-review
Originally posted by: Carl.D.S...@gmail.com
(No comment was entered for this change.)
Owner: Carl.D.S...@gmail.com
Status: Started
Originally posted by: ColinPKC...@gmail.com
(No comment was entered for this change.)
Labels: -Patch-review Patch-countdown
Originally posted by: Carl.D.S...@gmail.com
Issue 2100: Explanation of branches for CG
Explain branches, by Graham Percival
Changes to git commands by Carl Sorensen
http://codereview.appspot.com/5539062
Labels: Patch-new
Originally posted by: Carl.D.S...@gmail.com
Issue 2100: Explanation of branches for CG
Explain branches, by Graham Percival
Changes to git commands by Carl Sorensen
http://codereview.appspot.com/5539062
Originally posted by: lilypond...@gmail.com
Patchy the autobot says: LGTM.
Labels: Patch-review
Originally posted by: gra...@percival-music.ca
I think the notion of a "countdown" has played its part for this issue. I suggest that we push it; the only remaining quibbles might be over slight phrasing things that can be edited later if necessary/