-
Notifications
You must be signed in to change notification settings - Fork 11
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 PR template so users/maintainers can simply make the case for adding the badge #5
Comments
Can you elaborate a little more on what you mean here? Just a standard response that if maintainers are challenged, they can respond with rationale? |
I can do that; I have worked with templating issues and PRs on GitHub before. But I will need to know what exactly you want to populate it with. Example 🔗 Linked issue❓ Type of change
📚 Description📝 Checklist
PULL_REQUEST_TEMPLATE.md <!---
☝️ PR title should follow conventional commits (https://conventionalcommits.org)
-->
### 🔗 Linked issue
<!-- Please ensure there is an open issue and mention its number as #123 -->
### ❓ Type of change
<!-- What types of changes does your code introduce? Put an `x` in all the boxes that apply. -->
- [ ] 📖 Documentation (updates to the documentation, readme or JSDoc annotations)
- [ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
- [ ] 👌 Enhancement (improving an existing functionality like performance)
- [ ] ✨ New feature (a non-breaking change that adds functionality)
- [ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
- [ ] ⚠️ Breaking change (fix or feature that would cause existing functionality to change)
### 📚 Description
<!-- Describe your changes in detail -->
<!-- Why is this change required? What problem does it solve? -->
<!-- If it resolves an open issue, please link to the issue here. For example "Resolves #1337" -->
### 📝 Checklist
<!-- Put an `x` in all the boxes that apply. -->
<!-- If your change requires a documentation PR, please link it appropriately -->
<!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->
- [ ] I have linked an issue or discussion.
- [ ] I have updated the documentation accordingly. Credits for the sample nuxt-api-party |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
PR's come through for many reasons, some maintainers want to add the badge but are concerned about response from fellow maintainers. There should be a sample markdown that people can use to make the case in PRs.
The text was updated successfully, but these errors were encountered: