Comprehensive checklist for conducting thorough code reviews to ensure quality, security, and maintainability.
Don't run tests or linting, this is done automatically. Just inspect the code.
- Code does what it's supposed to do
- Edge cases are handled
- Error handling is appropriate
- No obvious bugs or logic errors
- Code is readable and well-structured
- Functions are small and focused
- Variable names are descriptive
- No code duplication
- Follows project conventions
- Check if new added dependencies are being used
- No obvious security vulnerabilities
- Input validation is present
- Sensitive data is handled properly
- No hardcoded secrets