Pull request: Difference between revisions

From ym2149.org
Jump to navigationJump to search
(Created page with "== Commit goals == * make life easier for the reviewer * separate refactoring from actual changes * make it easier to undo functionality in the future by reverting a commit * small enough number of commits to avoid crude merge-time squashing == Commit arrangement == * although in practice refactoring is done as needed, any such commits should go first Category:Programming")
 
No edit summary
 
(One intermediate revision by the same user not shown)
Line 1: Line 1:
Also see [[pull requests versus pair programming]].
== Commit goals ==
== Commit goals ==


Line 4: Line 5:
* separate refactoring from actual changes
* separate refactoring from actual changes
* make it easier to undo functionality in the future by reverting a commit
* make it easier to undo functionality in the future by reverting a commit
* small enough number of commits to avoid crude merge-time squashing
* small enough number of commits to avoid heavy handed merge-time squashing


== Commit arrangement ==
== Commit arrangement ==

Latest revision as of 11:30, 19 April 2024

Also see pull requests versus pair programming.

Commit goals

  • make life easier for the reviewer
  • separate refactoring from actual changes
  • make it easier to undo functionality in the future by reverting a commit
  • small enough number of commits to avoid heavy handed merge-time squashing

Commit arrangement

  • although in practice refactoring is done as needed, any such commits should go first