Skip to content

Commit 3bb615f

Browse files
author
Leonard Goodell
committed
ci: Update PR Template
Update to newly approved template Signed-off-by: Leonard Goodell <leonard.goodell@intel.com>
1 parent 8fadee7 commit 3bb615f

File tree

1 file changed

+17
-29
lines changed

1 file changed

+17
-29
lines changed

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 17 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -1,35 +1,23 @@
1-
## PR Checklist
2-
Please check if your PR fulfills the following requirements:
3-
4-
- [ ] Tests for the changes have been added (for bug fixes / features)
5-
- [ ] Docs have been added / updated (for bug fixes / features)
6-
7-
**If your build fails** due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-ui-go/blob/master/.github/Contributing.md.
8-
9-
## What is the current behavior?
10-
<!-- Please describe the current behavior and link to a relevant issue. -->
11-
12-
13-
## Issue Number:
1+
<!-- Expected Commit Message Description (imported automatically by GitHub) -->
2+
<!-- Must conform to [conventional commits guidelines](https://github.com/edgexfoundry/edgex-ui-go/blob/main/.github/Contributing.md) -->
3+
<!-- Expected Commit message must contain Closes/Fixes #IssueNumber statement when there is a related issue -->
144

5+
<!-- Add additional detailed description of need for change if no related issue -->
156

16-
## What is the new behavior?
7+
If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-ui-go/blob/main/.github/Contributing.md
178

9+
## PR Checklist
10+
Please check if your PR fulfills the following requirements:
1811

19-
## Does this PR introduce a breaking change?
20-
<!-- If this PR contains a breaking change, please describe the impact and migration path for existing applications below. -->
21-
22-
- [ ] Yes
23-
- [ ] No
24-
25-
## New Imports
26-
<!-- Are there any new imports or modules? If so, what are they used for and why? -->
27-
28-
- [ ] Yes
29-
- [ ] No
30-
31-
## Specific Instructions
32-
Are there any specific instructions or things that should be known prior to reviewing?
12+
- [ ] I am not introducing a breaking change (if you are, flag in conventional commit message with `BREAKING CHANGE:` describing the break)
13+
- [ ] I am not introducing a new dependency (add notes below if you are)
14+
- [ ] I have added unit tests for the new feature or bug fix (if not, why?)
15+
- [ ] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
16+
- [ ] I have opened a PR for the related docs change (if not, why?)
17+
<link to docs PR>
3318

19+
## Testing Instructions
20+
<!-- How can the reviewers test your change? -->
3421

35-
## Other information
22+
## New Dependency Instructions (If applicable)
23+
<!-- Please follow [vetting instructions](https://wiki.edgexfoundry.org/display/FA/Vetting+Process+for+3rd+Party+Dependencies) and place results here -->

0 commit comments

Comments
 (0)