Skip to content

Commit 5f81cc1

Browse files
authored
PYTHON-5401: Revise PR template for improved clarity and checks
1 parent c07c9b2 commit 5f81cc1

File tree

1 file changed

+4
-13
lines changed

1 file changed

+4
-13
lines changed

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
[Issue Key](https://jira.mongodb.org/browse/{ISSUE_KEY})
1+
[JIRA TICKET](https://jira.mongodb.org/browse/{ISSUE_KEY})
22

33
## Summary
44

@@ -12,28 +12,19 @@
1212

1313
<!-- How did you test the code? If you added unit tests, you can say that. If you didn’t introduce unit tests, explain why. All code should be tested in some way – so please list what your validation strategy was. -->
1414

15-
### Screenshots (optional)
16-
17-
<!-- Usually a great supplement to a test plan, especially if this requires local testing. -->
18-
1915
## Checklist
2016

2117
<!-- Do not delete the items provided on this checklist -->
2218

2319
### Checklist for Author
2420

25-
- [ ] Did you update the changelog (if necessary)?
26-
- [ ] Is the intention of the code captured in relevant tests?
27-
- [ ] If there are new TODOs, has a related JIRA ticket been created?
21+
- [ ] Does the title of the PR reference a JIRA Ticket?
22+
- [ ] Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?)
23+
- [ ] Is all relevant documentation (README or docstring) updated?
2824
- [ ] Has a MongoDB Employee run [the patch build of this PR](https://github.com/mongodb-labs/ai-ml-pipeline-testing?tab=readme-ov-file#running-a-patch-build-of-a-given-pr)?
2925

3026
### Checklist for Reviewer {@primary_reviewer}
3127

3228
- [ ] Does the title of the PR reference a JIRA Ticket?
3329
- [ ] Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?)
34-
- [ ] Have you checked for spelling & grammar errors?
3530
- [ ] Is all relevant documentation (README or docstring) updated?
36-
37-
## Focus Areas for Reviewer (optional)
38-
39-
<!-- List any complex portion of code you believe needs particular scrutiny and explain why. -->

0 commit comments

Comments
 (0)