Skip to content

Commit

Permalink
Further PR template updates (#662)
Browse files Browse the repository at this point in the history
  • Loading branch information
forsyth2 authored Jan 17, 2025
1 parent 338ca42 commit 0da5fdf
Showing 1 changed file with 12 additions and 8 deletions.
20 changes: 12 additions & 8 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,12 @@
## Issue resolution
## Summary

Objectives:
- Objective 1
- Objective 2
- ...
- Objective n

Issue resolution:
- Closes #<ISSUE_NUMBER_HERE>

Select one: This pull request is...
Expand All @@ -7,24 +15,20 @@ Select one: This pull request is...
- [ ] a new feature: increment the minor version
- [ ] an incompatible (non-backwards compatible) API change: increment the major version

Please fill out either the "Small Change" or "Big Change" section, and delete the other.
Please fill out either the "Small Change" or "Big Change" section (the latter includes the numbered subsections), and delete the other.

## Small Change

- [ ] To merge, I will use "Squash and merge". That is, this change should be a single commit.
- [ ] Logic: I have visually inspected the entire pull request myself.
- [ ] Pre-commit checks: All the pre-commits checks have passed.

## Big Change

- [ ] To merge, I will use "Create a merge commit". That is, this change is large enough to require multiple units of work (i.e., it should be multiple commits).

### 1. Does this do what we want it to do?

Objectives:
- Objective 1
- Objective 2
- ...
- Objective n

Required:
- [ ] Product Management: I have confirmed with the stakeholders that the objectives above are correct and complete.
- [ ] Testing: I have added or modified at least one "min-case" configuration file to test this change. Every objective above is represented in at least one cfg.
Expand Down

0 comments on commit 0da5fdf

Please sign in to comment.