Skip to content
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

Add initial pull request template #3208

Merged
merged 2 commits into from
Dec 13, 2018
Merged

Add initial pull request template #3208

merged 2 commits into from
Dec 13, 2018

Conversation

ddbeck
Copy link
Collaborator

@ddbeck ddbeck commented Dec 13, 2018

This PR aims adds a little pull request template to help catch a few things that increase friction on PR reviews (e.g., no description, no background info, linter problems, etc.).

@ddbeck ddbeck added infra 🏗️ Infrastructure issues (npm, GitHub Actions, releases) of this project docs ✍️ Issues or pull requests regarding the documentation of this project. labels Dec 13, 2018
@ddbeck ddbeck requested a review from Elchi3 December 13, 2018 15:53
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Daniel, I think this will be very useful! 👍

One comment on the part where we ask about providing evidence or proofs. I thought that one is the most important and I think we want to make it very clear.

@@ -0,0 +1,6 @@
A checklist to help your pull request get merged faster:
- [ ] Summarize your changes
- [ ] Data: link to supporting info (examples: docs, changelogs, source control, bug trackers, tests)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if it is crystal clear what we mean with "supporting info" here. Maybe:

- [ ] Provide resources that help verifying support information (links to the browser's docs, changelogs, source control, bug trackers, tests)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea. Pushed another version of this!

Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this, @ddbeck! 👍

We can improve it as we see it used in the wild. This is a good start!

@Elchi3 Elchi3 merged commit 5c7b25c into mdn:master Dec 13, 2018
@ddbeck ddbeck deleted the pr-template branch December 13, 2018 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs ✍️ Issues or pull requests regarding the documentation of this project. infra 🏗️ Infrastructure issues (npm, GitHub Actions, releases) of this project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants