Task #5641
Update PR checklist to cover actual needs
Status:
NEW
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:
0%
Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Documentation
Sprint:
Quarter:
Description
There is a checklist in our docs [0] but doesn't cover towncrier and possibility to update auto-generated API docs.
[0] https://docs.pulpproject.org/en/3.0/nightly/contributing/pull-request-walkthrough.html#checklist
History
#1
Updated by daviddavis about 1 year ago
The checklist does cover towncrier. It's number 4. We should call out docstring in the checklist though.
Also, in the conversation that motivated this issue, there was mention of https://devchecklists.com/. They have a PR checklist:
https://devchecklists.com/pull-requests-checklist/
Along with some others. We could maybe check to see if we're missing anything.
Please register to edit this issue
.