News for hackers. Stuff that matters.

A comment on Git Reflow. Parent comment

Re:Git Reflow by ghayes on Thursday May 19, 2016 @08:35PM (#11734910)
I agree here. Be a good steward of your commits, and let `rebase -i` be your tool for that. A bunch of "err, try this instead" should be washed away, but it doesn't help to commit "Add Huge Feature" +10,000/-2,000 because "I should squash my feature"
[ Parent | on HN ]
Re:Git Reflow by mikepurvis on Thursday May 19, 2016 @09:23PM (#11735066)
If you're putting up 10k lines of code in a single review, you're doing it wrong anyway. Why wouldn't you have multiple reviews that each add different parts of a feature and link to the same ticket?
[ Parent | on HN ]
Re:Git Reflow by adrianratnapala on Friday May 20, 2016 @03:26AM (#11736058)
If I wanted to review 200 lines of changes, then I would still prefer to see it as a series of patches, each maybe with 1-20 lines changed and each with a clear purpose.
[ Parent | on HN ]