Skip to content
Merged
Show file tree
Hide file tree
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
73 changes: 73 additions & 0 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,77 @@ If no user-facing changelog entry is needed, apply the skip-changelog label to t

---

## Risk Assessment

<!--
Evaluate the risk level of this PR and potential impact on production.
This helps reviewers prioritize and determine the appropriate review rigor.
-->

**Risk Level:** <!-- Choose one: Low | Medium | High | Critical -->

**Potential Impact:**
<!-- Describe what could go wrong if this change has bugs. Examples:
- Low: Typo in documentation, minor style tweak
- Medium: New feature in isolated component, refactor with test coverage
- High: Changes to core authentication, database schema changes
- Critical: Payment processing, security fixes, data migrations
-->

**Mitigation Steps:**
<!-- What steps have been taken to reduce risk? Examples:
- Comprehensive test coverage (unit, integration, E2E)
- Feature flag for gradual rollout
- Tested in staging environment
- Reversible database migration
- Monitoring and alerting in place
-->

---

## How to Test

<!--
Provide clear, step-by-step instructions for testing this PR.
Include:
- Prerequisites (data setup, feature flags, environment variables)
- User flows to test
- Expected results for each step
- Edge cases to verify
- Screenshots or videos demonstrating the changes (for UI changes)
-->

### Prerequisites

<!-- Example:
- Create a test user account
- Enable feature flag: `FEATURE_NEW_DASHBOARD=true`
- Seed database with sample data: `npm run db:seed`
-->

### Test Steps

1. **Step 1:** <!-- Describe action and expected result -->
2. **Step 2:** <!-- Describe action and expected result -->
3. **Step 3:** <!-- Describe action and expected result -->

### Expected Results

<!-- Describe what success looks like. Include screenshots or videos for UI changes. -->

### Edge Cases to Verify

<!-- List edge cases that should be tested. Examples:
- [ ] Empty state (no data)
- [ ] Maximum data (performance test)
- [ ] Error handling (network failure, invalid input)
- [ ] Mobile/responsive design
- [ ] Accessibility (keyboard navigation, screen reader)
- [ ] Browser compatibility (Chrome, Firefox, Safari)
-->

---

### Checklist (Global DoD / PR)

- [ ] All AC met and demonstrated
Expand All @@ -69,6 +140,8 @@ If no user-facing changelog entry is needed, apply the skip-changelog label to t
- [ ] Security/perf impact reviewed where relevant
- [ ] Code/design reviews approved
- [ ] CI green; linked issues closed; release notes prepared (if shipping)
- [ ] Risk assessment completed above
- [ ] Testing instructions provided above

---

Expand Down
Loading
Loading