Compare commits
5 Commits
master
...
PYTHON-540
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
1a640b11a5 | ||
|
|
10ba0cf28a | ||
|
|
fb4b2c2c65 | ||
|
|
efe4893e98 | ||
|
|
97b0f0c859 |
27
.github/pull_request_template.md
vendored
27
.github/pull_request_template.md
vendored
@ -1,38 +1,33 @@
|
|||||||
<!-- Thanks for contributing! -->
|
<!-- Thanks for contributing! -->
|
||||||
<!-- Please ensure that the title of the PR is in the following form:
|
<!-- Please ensure that the title of the PR is in the following form:
|
||||||
[Issue Type]-[Issue Key]: Issue Title
|
[JIRA TICKET]: Issue Title
|
||||||
|
|
||||||
If you are an external contributor and there is no JIRA ticket associated with your change, then use your best judgement
|
If you are an external contributor and there is no JIRA ticket associated with your change, then use your best judgement
|
||||||
for the PR title. A MongoDB employee will create a JIRA ticket and edit the name and links as appropriate.
|
for the PR title. A MongoDB employee will create a JIRA ticket and edit the name and links as appropriate.
|
||||||
|
|
||||||
|
Note on AI Contributions:
|
||||||
|
We do not accept pull requests that are primarily or substantially generated by AI tools (ChatGPT, Copilot, etc.).
|
||||||
|
All contributions must be written and understood by human contributors.
|
||||||
-->
|
-->
|
||||||
[Issue Key](https://jira.mongodb.org/browse/%7BISSUE_KEY%7D)
|
[JIRA TICKET]
|
||||||
## Summary
|
|
||||||
<!-- What conceptually is this PR introducing? If context is already provided from the JIRA ticket, still place it in the
|
|
||||||
Pull Request as you should not make the reviewer do digging for a basic summary. -->
|
|
||||||
|
|
||||||
## Changes in this PR
|
## Changes in this PR
|
||||||
<!-- What changes did you make to the code? What new APIs (public or private) were added, removed, or edited to generate
|
<!-- What changes did you make to the code? What new APIs (public or private) were added, removed, or edited to generate
|
||||||
the desired outcome explained in the above summary? -->
|
the desired outcome explained in the above summary? -->
|
||||||
|
|
||||||
## Testing Plan
|
## Test Plan
|
||||||
<!-- How did you test the code? If you added unit tests, you can say that. If you didn’t introduce unit tests, explain why.
|
<!-- 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. -->
|
All code should be tested in some way – so please list what your validation strategy was. -->
|
||||||
|
|
||||||
### Screenshots (optional)
|
|
||||||
<!-- Usually a great supplement to a test plan, especially if this requires local testing. -->
|
|
||||||
|
|
||||||
## Checklist
|
## Checklist
|
||||||
<!-- Do not delete the items provided on this checklist. -->
|
<!-- Do not delete the items provided on this checklist. -->
|
||||||
|
|
||||||
### Checklist for Author
|
### Checklist for Author
|
||||||
- [ ] Did you update the changelog (if necessary)?
|
- [ ] Did you update the changelog (if necessary)?
|
||||||
- [ ] Is the intention of the code captured in relevant tests?
|
- [ ] Is there test coverage?
|
||||||
- [ ] If there are new TODOs, has a related JIRA ticket been created?
|
- [ ] Is any followup work tracked in a JIRA ticket? If so, add link(s).
|
||||||
|
|
||||||
### Checklist for Reviewer {@primary_reviewer}
|
### Checklist for Reviewer
|
||||||
- [ ] Does the title of the PR reference a JIRA Ticket?
|
- [ ] Does the title of the PR reference a JIRA Ticket?
|
||||||
- [ ] Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?)
|
- [ ] Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?)
|
||||||
- [ ] Have you checked for spelling & grammar errors?
|
|
||||||
- [ ] Is all relevant documentation (README or docstring) updated?
|
- [ ] Is all relevant documentation (README or docstring) updated?
|
||||||
|
|
||||||
## Focus Areas for Reviewer (optional)
|
|
||||||
<!-- List any complex portion of code you believe needs additional scrutiny and explain why. -->
|
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user