quill: add code review checklist
This commit is contained in:
1 parent
8c48c156ce
commit
20bec6d915
1 file changed
+21
@@ -0,0 +1,21 @@
|
|||||||
|
# Code Review Checklist
|
||||||
|
|
||||||
|
## Security
|
||||||
|
- No secrets in code or configs.
|
||||||
|
- Input validation and sanitization.
|
||||||
|
- Authentication and authorization checks.
|
||||||
|
|
||||||
|
## Performance
|
||||||
|
- Avoid N+1 queries.
|
||||||
|
- Cache where appropriate.
|
||||||
|
- Profile before optimizing.
|
||||||
|
|
||||||
|
## Testing
|
||||||
|
- Unit tests for new logic.
|
||||||
|
- Integration tests for APIs.
|
||||||
|
- Tests pass locally and in CI.
|
||||||
|
|
||||||
|
## Documentation
|
||||||
|
- README updated if behavior changes.
|
||||||
|
- Inline comments for non-obvious code.
|
||||||
|
- Changelog entry added.
|
||||||
Reference in new issue
Block a user