Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
124 changes: 124 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,124 @@
## Description
<!-- Provide a brief description of the changes in this PR -->

## Type of Change
<!-- Mark the relevant option with an 'x' -->

- [ ] Bug fix (non-braking change that fixes an issue)
- [ ] New feature (non-breaking change that adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] Documentation update
- [ ] Code refactoring
- [ ] Performance improvement
- [ ] Test addition or update
- [ ] Build/CI update

## Related Issues
<!-- Link to related issues using #issue_number -->

Closes #

## Changes Made
<!-- List the main changes made in this PR -->

-
-
-

## Screenshots/Videos
<!-- If applicable, add screenshots or videos demonstrating the changes -->

### Before
<!-- Screenshot of before changes -->

### After
<!-- Screenshot of after the changes -->

## Testing
<!-- Describe the tests you ran and how to reproduce them -->

### Test Configuration
- **OS:**
- **Browser:**
- **Node version:**

### Test Steps
1.
2.
3.

## Checklist
<!-- Mark completed items with an 'x' -->

### Code Quality
- [ ] My code follows the project's style guidelines
- [ ] I have performed a self-review of my code
- [ ] I have commented my code, particulary 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

### Testing
- [ ] I have tested on desktop
- [ ] I have tested on mobile
- [ ] I have tested on tablet
- [ ] I have tested on multiple browseers (Chrome, Firefox, Safari)
- [ ] All tests pass: `npm test`
- [ ] TypeScript compiles: `npm run typecheck`
- [ ] Linting passes: `npm run lit`

### Accessibility
- [ ] I have tested with a screen reader
- [ ] All images have alt text
- [ ] Color contrast meets WCAG AA standards
- [ ] Keyboard navigation works
- [ ] Focus indicators are visible

### Responsive Design
- [ ] Works on mobile (375px width)
- [ ] Works on tablet (768px width)
- [ ] Works on desktop (1440px width)
- [ ] No horizontal scrolling on mobile

### Performance
- [ ] Images are optimized
- [ ] No unnecessary re-renders
- [ ] Bundle size impact is reasonable
- [ ] Page load time is acceptable

### Security
- [ ] No sensitive data in code
- [ ] API keys are in environment variables
- [ ] Input validation is implemented
- [ ] SQL injection prevention is in place (if applicable)
- [ ] XSS prevention is in place

### Documentation
- [ ] README updated (if needed)
- [ ] API documentation updated (if needed)
- [ ] Component props documented (if new component)
- [ ] CHANGELOG updated (if applicable)

## Additional Notes
<!-- Add any additional notes for reviewers -->

## For Reviewers
<!-- What should reviewers focus on? -->

**Focus areas:**
-
-

**Questions:**
-
-

---

**Checklist for Reviewers**
- [ ] Code follows project conventions
- [ ] Changes are well-tested
- [ ] Documentation is clear and complete
- [ ] No obvious security issues
- [ ] Performance impact is acceptable
Loading