[Proposal] Add a PR template to the Django repo

61 views
Skip to first unread message

Sarah Boyce

unread,
Apr 7, 2022, 8:19:44 AM4/7/22
to Django developers (Contributions to Django itself)
Hi all,

Wondering if I could get some consensus at to whether a PR template would be beneficial to the Django GitHub repository.

What I have in mind, would be pulling elements from the patch review checklist[1] to be an actual checklist[2] (perhaps in a collapsed section[3]) that appears on the PR itself.

Reasons I think it's not a good idea:
- we would be duplicating information in 2 places (in the docs and in the PR template), which would be harder to maintain.
- when creating your first PR, a comment asks you to review this checklist (links to the docs). So people without this comment have done it before and presumably read it.

Reasons I think it's a good idea:
- brings the information needed closer to the pr author, without linking them away
- I had "read" the checklist, and have since re-read, and there was a lot I hadn't taken in. I think it is still useful for after the first patch.
- I like to cross things off lists

I am on the fence as to whether it would add value or not. If we think it would add value, I would be happy to create a ticket in Track and raise a PR.

Looking forward to hearing your thoughts.
Have a wonderful day.
Sarah


Note: I can see a PR template was briefly mentioned before[4] when discussing adding a pre-commit but I can't mind other mentions so far. Sorry if it has already been discussed.


Reply all
Reply to author
Forward
0 new messages