Settings

Theme

How to perform code reviews like a senior software engineer

newsletter.beginner.dev

1 points by dglass 3 years ago · 1 comment

Reader

alanwreath 3 years ago

Maybe I've missed the part in this document where it's referenced but: Automate. That's one part of the PR process that is crucial to performing code reviews like a Senior. In other words the PR process should start before engineers are even able to see the code. Nobody wants to talk about style, code smells, and code that might be broken. Linters. Formatters. Tests. That's what makes a code review a "senior" experience. So, as a senior, if the tooling isn't there to set up code reviews for success, then advocate for it.

Keyboard Shortcuts

j
Next item
k
Previous item
o / Enter
Open selected item
?
Show this help
Esc
Close modal / clear selection