PULL_REQUEST_TEMPLATE.md•3.43 kB
# Pull Request
## Summary
<!-- Provide a brief description of your changes -->
## Type of Change
<!-- Check the appropriate box -->
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] Performance improvement
- [ ] Documentation update
- [ ] Code refactoring (no functional changes)
- [ ] Test improvements
- [ ] CI/CD improvements
## Related Issues
<!-- Link to related issues using keywords like "Fixes #123" or "Closes #456" -->
- Fixes #
- Related to #
## Changes Made
<!-- Describe the changes you made in detail -->
-
-
-
## Testing
<!-- Describe how you tested your changes -->
- [ ] Unit tests added/updated
- [ ] Integration tests added/updated
- [ ] Manual testing performed
- [ ] All existing tests pass
### Test Results
<!-- Include relevant test output or screenshots -->
```
# Paste test results here
```
## Performance Impact
<!-- Describe any performance implications -->
- [ ] No performance impact
- [ ] Performance improved
- [ ] Performance may be affected (please explain)
**Performance Notes:**
<!-- If applicable, include benchmarks or performance analysis -->
## Breaking Changes
<!-- List any breaking changes and migration instructions -->
- [ ] No breaking changes
- [ ] Breaking changes present (list below)
**Breaking Changes:**
<!-- Describe breaking changes and how to migrate -->
## Documentation
<!-- Check all that apply -->
- [ ] Code is self-documenting
- [ ] Added/updated JSDoc comments
- [ ] Updated README.md
- [ ] Updated API documentation
- [ ] Added/updated examples
- [ ] No documentation changes needed
## Code Quality
<!-- Confirm code quality checks -->
- [ ] Code follows the project's style guidelines
- [ ] Self-review of code completed
- [ ] Code is properly commented
- [ ] No console.log statements left in code
- [ ] Error handling is appropriate
- [ ] TypeScript types are properly defined
## Security Considerations
<!-- Check if applicable -->
- [ ] No security implications
- [ ] Security review completed
- [ ] Added input validation
- [ ] No sensitive data exposed
- [ ] API keys/secrets properly handled
## Checklist
<!-- Check all items before submitting -->
- [ ] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] I have run `npm run lint` and fixed any issues
- [ ] I have run `npm run typecheck` and resolved any type errors
- [ ] I have run `npm run format` to ensure consistent code formatting
- [ ] Any dependent changes have been merged and published
## Screenshots/Media
<!-- If applicable, add screenshots or other media to help explain your changes -->
## Additional Notes
<!-- Add any additional notes for reviewers -->
---
**Reviewer Guidelines:**
- Verify that all tests pass
- Check for proper error handling
- Ensure code follows project conventions
- Validate that documentation is updated if needed
- Consider performance implications
- Review security aspects if applicable
🤖 Generated with [Claude Code](https://claude.ai/code)