-
Notifications
You must be signed in to change notification settings - Fork 139
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
41c9955
commit f974104
Showing
1 changed file
with
12 additions
and
27 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 |
|---|---|---|
| @@ -1,37 +1,22 @@ | ||
| If you haven't already, please review our [contributing guidelines](CONTRIBUTING.md) before opening your pull request. | ||
|
|
||
| Use this template to give a detailed message describing the change you want to make to the code. | ||
|
|
||
| The title of this pull request should be a brief "purpose" for this change. | ||
|
|
||
| --- Delete this line and those above before hitting "Create pull request" --- | ||
|
|
||
| TYPE: choose one of [bug fix, enhancement, new feature, feature removed, no impact, text only] | ||
|
|
||
| KEYWORDS: approximately 3 to 6 words (more is always better) related to your commit, separated by commas | ||
|
|
||
| SOURCE: Developer's name and affiliation | ||
| SOURCE: Developer's name and affiliation | ||
|
|
||
| DESCRIPTION OF CHANGES: One or more paragraphs describing problem, solution, and required changes. | ||
|
|
||
| ISSUE: The GitHub Issue that this PR addresses. For issue number 123, it would be: | ||
| ``` | ||
| Fixes #123 | ||
| ``` | ||
|
|
||
| TESTS CONDUCTED: Explicitly state if regression, integration, or unit tests were run before submitting (do not include the CI tests automatically run by GitHub) | ||
|
|
||
| NOTES: Optional, as appropriate. Delete if not used. Included only once for new features requiring several merge cycles. Changes to default behavior are also note worthy. | ||
| ISSUE: The GitHub Issue that this PR addresses. For issue number 123, it would be `Fixes #123` | ||
|
|
||
| TESTS CONDUCTED: Explicitly state if regression, integration, or unit tests were run before submitting. | ||
|
|
||
| ### Checklist | ||
| Merging the PR depends on following checklist being completed. Add `X` between each of the square | ||
| brackets if they are completed in the PR itself. If a bullet is not relevant to you, please comment | ||
| on why below the bullet. | ||
| NOTES: Optional, as appropriate. Delete if not used. | ||
|
|
||
| - [ ] Closes issue #xxxx (An issue must exist or be created to be closed. The issue describes and documents the problem and general solution, the PR describes the technical details of the solution.) | ||
| - [ ] Tests added (unit tests and/or regression/integration tests) | ||
| - [ ] Backwards compatible | ||
| - [ ] Requires new files? If so, how to generate them. | ||
| - [ ] Documentation included | ||
| - [ ] Short description in the Development section of `NEWS.md` | ||
| <!-- | ||
| ITEMS THAT MIGHT BE USEFUL TO CONSIDER | ||
| - Closes issue #xxxx | ||
| - Tests added (unit tests and/or regression/integration tests) | ||
| - Backwards compatible | ||
| - Documentation included | ||
| - Short description in the Development section of `NEWS.md` | ||
| ---> |