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.