Update PULL_REQUEST_TEMPLATE.md
This commit is contained in:
26
.github/PULL_REQUEST_TEMPLATE.md
vendored
26
.github/PULL_REQUEST_TEMPLATE.md
vendored
@@ -1,27 +1,31 @@
|
||||
## Before creating a pull request, please check if you have met all required steps
|
||||
# Before creating your Pull Request...
|
||||
|
||||
**Is this a fix or an improvement?**
|
||||
- New Pull Requests should include a good description of what's being merged.
|
||||
- Ideally, all Pull Requests are preceded by a discussion initiated in an Issue on this repository.
|
||||
- For bug fixes is mandatory to have tests that cover and fail when the bug is present and will pass after this Pull Request.
|
||||
- For changes and improvements, new tests have to be provided to cover the new features.
|
||||
|
||||
|
||||
## What does this change implement/fix?
|
||||
|
||||
... *answer here*
|
||||
|
||||
## Is this a fix or an improvement?
|
||||
|
||||
- [ ] Fix
|
||||
- [ ] Improvement
|
||||
|
||||
**What does this change implement/fix?**
|
||||
|
||||
...
|
||||
|
||||
|
||||
**Have discussed this change in an issue?**
|
||||
## Have discussed this change in an issue?
|
||||
|
||||
- [ ] Yes
|
||||
- [ ] No
|
||||
|
||||
**Was some test failing because of this issue or change needed?**
|
||||
## Was some test failing because of this issue or change needed?
|
||||
|
||||
- [ ] Yes
|
||||
- [ ] No
|
||||
|
||||
**Are you including tests to cover this change?**
|
||||
## Are you including tests to cover this change?
|
||||
|
||||
- [ ] Yes
|
||||
- [ ] No
|
||||
|
||||
|
||||
Reference in New Issue
Block a user