Pre-merge Code Reviews
Rules, benefits, patterns, and anti-patterns for reviewing code before merging it.
5
http://verraes.net/2013/10/pre-merge-code-reviews/
3 months ago

I would probably add to the "Making it easier" checklist to use a linter that implements your team style guide so that you easily avoid "Focusing on junk details". You can also embed it in a test and add a git hook that does not allow to push if any test fails, it helps make the codebase consistent among the team ;)

3 months ago

👍 for the pre-push hook :)

Cancel
Submit your answer