1.4 KiB
1.4 KiB
Pull Request
Summary
Changes
Related Issue
Closes #
Checklist
Git Workflow
- External contributors: Followed Fork → topic branch → upstream PR flow
- Collaborators: Used topic branch (no direct commits to main)
- Rebased on upstream/main (
git rebase upstream/main, no conflicts) - Commit messages follow Conventional Commits (
feat:,fix:,docs:, etc.)
Code Quality
- Changes are limited to a single purpose (not a mega-PR; aim for ~200 lines diff)
- Follows existing code conventions and patterns
- Added appropriate tests for new features/fixes
- Lint/Format/Typecheck all pass
- CI/CD pipeline succeeds (green status)
Security
- No secrets or credentials committed
- Necessary files excluded via
.gitignore - No breaking changes, or if so:
!commit + MIGRATION.md documented
Documentation
- Updated documentation as needed (README, CLAUDE.md, docs/, etc.)
- Added comments for complex logic
- API changes are properly documented