Project

Profile

Help

Task #5641

Update PR checklist to cover actual needs

Added by ppicka about 1 month ago. Updated about 1 month ago.

Status:
NEW
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:

0%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Documentation
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:

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 month 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

Also available in: Atom PDF