Expanded checklist for pull request submissions in VerifyWise.
- I have performed a self-review of my code
- My code follows the project's style guidelines
- I have removed all debugging code (console.log, debugger, etc.)
- I have removed all commented-out code
- Variable and function names are clear and descriptive
- Complex logic has explanatory comments
- I have deployed and tested the code locally
- The feature/fix works as expected
- Edge cases have been considered and handled
- Error states are handled gracefully
- I have written tests for new functionality
- All existing tests pass
- Test coverage meets minimum requirements (80%)
- Tests are meaningful (not just for coverage)
- User input is validated and sanitized
- No sensitive data is logged or exposed
- Authentication/authorization is properly implemented
- No hardcoded secrets or credentials
- SQL injection prevention verified (parameterized queries)
- No unnecessary database queries
- No N+1 query issues
- Large data sets are paginated
- No expensive operations in loops
- UI elements use theme references (not hardcoded values)
- Font sizes reference theme typography
- Colors reference theme palette
- Component is responsive
- Accessibility requirements met (ARIA labels, keyboard navigation)
- Screenshots/video attached for UI changes
- Code is self-documenting with clear names
- Complex logic is commented
- Public APIs have JSDoc/docstrings
- README updated if needed
- Commit messages follow conventional format
- Branch is up to date with target branch
- PR title follows format:
type(scope): description - PR description is complete and clear
- Issue number is linked (Fixes #xxx)
- Issue is assigned to me
- PR is labeled correctly
- PR addresses a single, focused feature
- No unrelated changes included
type(scope): short description
Types: feat, fix, docs, style, refactor, test, chore
type(scope): subject
[body]
[footer]
- Aim for < 400 lines of changes
- Split large PRs into smaller ones
- ✅ All items above checked
- ✅ CI/CD pipeline passes
- ✅ Self-review completed
- ✅ PR description filled out
- ✅ Screenshots attached (for UI)