-
Notifications
You must be signed in to change notification settings - Fork 70
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Create PULL_REQUEST_TEMPLATE.md (#211)
Signed-off-by: Yuwei Ba <[email protected]>
- Loading branch information
Showing
1 changed file
with
49 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,49 @@ | ||
<!-- | ||
First of all, thank you for your contribution! 😄 | ||
For requesting to pull a new feature or bugfix, please send it from a feature/bugfix branch based on the `master` branch. | ||
Before submitting your pull request, please make sure the checklist below is confirmed. | ||
Your pull requests will be merged after one of the collaborators approve. | ||
Thank you! | ||
--> | ||
|
||
### 🤔 This is a ... | ||
|
||
- [ ] New feature | ||
- [ ] Bug fix | ||
- [ ] Performance optimization | ||
- [ ] Enhancement feature | ||
- [ ] Refactoring | ||
- [ ] Code style optimization | ||
- [ ] Test Case | ||
- [ ] Branch merge | ||
- [ ] Workflow | ||
- [ ] Other (about what?) | ||
|
||
### 🔗 Related issue link | ||
|
||
<!-- | ||
1. Put the related issue or discussion links here. | ||
2. close #xxxx or fix #xxxx for instance. | ||
--> | ||
|
||
### 💡 Background and solution | ||
|
||
<!-- | ||
1. Describe the problem and the scenario. | ||
2. Provider a sample config if you can. | ||
3. How to fix the problem, and list the final implementation and usage sample if that is a new feature. | ||
--> | ||
|
||
### 📝 Changelog | ||
|
||
<!-- | ||
Describe changes from the user side, and list all potential break changes or other risks. | ||
---> | ||
|
||
|
||
### ☑️ Self-Check before Merge | ||
|
||
⚠️ Please check all items below before requesting a reviewing. ⚠️ | ||
|
||
- [ ] Doc is updated/provided or not needed | ||
- [ ] Changelog is provided or not needed |