DEV Community

ke jia
ke jia

Posted on

My Code Review Checklist: 7 Things I Check Before Every PR

#codereview #beginners #github #javascript

After reviewing 100+ PRs this year, here's my 7-point checklist:

1. Secrets 🔒

npx @wuchunjie/dotguard
Enter fullscreen mode Exit fullscreen mode

2. Hotspots 🔥

npx @wuchunjie/gitpulse --diff HEAD~1
Enter fullscreen mode Exit fullscreen mode

3. Structure 🏗️

npx scaffoldx-cli audit --compare origin/main
Enter fullscreen mode Exit fullscreen mode

4. Tests — Do they fail when code breaks?
5. Error Handling — No silent failures
6. Naming — If it needs a comment, rename it
7. Performance — N+1 queries? Unnecessary deps?

Quick CLI flow:

git checkout pr-branch
dotguard && gitpulse --diff main..pr-branch && scaffoldx-cli audit
Enter fullscreen mode Exit fullscreen mode

All 3 pass → approve.

☕ Ko-fi

Top comments (0)