-
Notifications
You must be signed in to change notification settings - Fork 370
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PR template #671
Comments
Sounds like a good idea. Do you want to propose one? My only experience/recommendation is that we try and keep it short and not include things in a checklist that CI can/does check for us, let's bundle those as "make sure the checks all pass" if we feel that would help people. On a previous project we had a (at the end when we decided to remove it) checklist of seven or so items and most of the time people ignored the whole list :-/ |
I opened a similar issue on team-compass for an org-wide template jupyterhub/team-compass#302 |
This also slipped my to-do list. Want to work on this together @manics ? |
Definitely! How about if I open a PR on https://github.com/jupyterhub/.github/ based on the comments on jupyterhub/team-compass#302 and we go from there? |
Hey 👋🏼
Proposed change
With the recent merge of #654 and #655 I think we should also include templates for PR
Why?
Having standard templates will not only make it easier for those reviewing them to track the state of the PR but also will encourage the submitters to make sure they adhere to the contributing guidelines and make some checks before the PR is sent
Alternative options
--
Who would use this feature?
Contributors to the project
How much effort will adding it take?
Who can do this work?
Low effort - anyone could take this as this is a non-coding activity.
I can even take this as I already have some templates that I have for personal use/projects
The text was updated successfully, but these errors were encountered: