It’s easy to overlook issues in code reviews
It’s easy to overlook issues in code reviews.
So, I use a high-level checklist:
DRY
Usable
Secure
Tested
Robust
Readable
Accessible
Performant
Configurable
Documented
Meets requirements
I automate every check I can. This list can’t be automated.
So, I run the code. 👍
🧵
This list is short enough that I can use it consistently. But long enough to be thorough.
I use it to look at the code through different “lenses”. It prompts me to think about these important concerns.
The checklist assures I consider all the important aspects in my review.
I store this checklist in PULL_REQUEST_TEMPLATE.md.
GitHub automatically reads this file and injects the checklist into each PR. 🔥
