Skip to content

Commit a9238b4

Browse files
Update PR Template (RooCodeInc#4175)
* update pr template * change wording
1 parent 79edbf9 commit a9238b4

File tree

1 file changed

+47
-3
lines changed

1 file changed

+47
-3
lines changed

.github/pull_request_template.md

Lines changed: 47 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,46 @@
1+
<!-- ⚠️ Important: Discussion Required Before PR (Community Contributors) -->
2+
3+
**For community contributors, before submitting this PR, please ensure you have:**
4+
5+
- [ ] **Opened an issue** to discuss your proposed changes with the community
6+
- [ ] **Received approval** from a core Cline contributor to proceed with the implementation
7+
- [ ] **Linked the issue below** in the "Related Issue" section
8+
9+
**Exceptions:** Small bug fixes, typo corrections, minor wording improvements, or simple type fixes that don't change functionality may be submitted directly.
10+
11+
**Why this requirement?** We deeply appreciate all community contributions - they are the core reason we're able to operate successfully and keep innovating! We welcome community input and want to make it as easy as possible for people to submit quality work. This process helps our core maintainers review new ideas faster and saves contributor time by ensuring you have the go-ahead before spending time on implementation.
12+
13+
---
14+
15+
### Related Issue
16+
17+
<!-- Replace XXXX with the issue number that this PR addresses -->
18+
**Issue:** #XXXX
19+
120
### Description
221

3-
<!-- Describe your changes in detail. What problem does this PR solve? -->
22+
<!--
23+
Help reviewers understand your changes by making this PR readable and well-organized:
24+
25+
- What problem does this PR solve?
26+
- Why were these changes introduced and what purpose do they serve?
27+
- For larger changes, provide context about your approach and reasoning
28+
29+
Small PRs may need minimal description, but larger changes benefit from explaining where you're coming from. Much of this context can be in the linked issue above, so feel free to reference it rather than repeating everything here.
30+
-->
431

532
### Test Procedure
633

7-
<!-- How did you test this? Are you confident that it will not introduce bugs? If so, why? -->
34+
<!--
35+
Please walk us through your testing approach and thought process. This helps reviewers understand that you've thoroughly considered the impact of your changes:
36+
37+
- How did you test this change?
38+
- What could potentially break and how did you verify it doesn't?
39+
- What existing functionality might be affected and how did you check it still works?
40+
- Why are you confident this is ready for merge?
41+
42+
We're not looking for exhaustive documentation - just evidence that you've thought through the implications of your changes and tested accordingly.
43+
-->
844

945
### Type of Change
1046

@@ -29,7 +65,15 @@
2965

3066
### Screenshots
3167

32-
<!-- For UI changes, add screenshots here -->
68+
<!--
69+
Help reviewers quickly understand your changes:
70+
71+
- **UI Changes**: Please include screenshots showing before/after states
72+
- **Complex Workflows**: Consider uploading a screen recording (video) if your changes involve multiple steps or state transitions
73+
- **Backend Changes**: Not required, but feel free to include terminal output or other evidence that demonstrates functionality
74+
75+
This helps reviewers see what you've built without having to pull down and test your branch first.
76+
-->
3377

3478
### Additional Notes
3579

0 commit comments

Comments
 (0)