DEV Community

loading...

Discussion on: How to make great code reviews? 🤝

Collapse
nbull92 profile image
NBull92

I don't neccassarily have one for the team, but I follow one that I found a while ago (it might even be from another dev.to post):

Summarize the change in less than 50 characters.

Because:

  • Explain the reasons you made this change
  • Make a new bullet point for each reason
  • Each line should be under 72 characters

Explain what was done in this commit with more depth than the 50 character subject line. Remember to at 72 characters

Include additional notes, relevant links, or co-authors/reviewers