DEV Community

loading...

Discussion on: My code review checklist

Collapse
silentsudo profile image
Ashish Agre

Code Formatting: one of the main I would consider, often when a new developer joins in he has his own settings, fixes 1 bug and boom you seem a lot much change in PR.

Collapse
geekgalgroks profile image
Jenn Author

That is where linters come in handy. You have the debate over code standards once and then put those rules into a linter.

It feels less mean having a computer yell at you about spacing versus a person reviewing code.