From d3490e543506d86922a31a5f315f7cbbbcbe835c Mon Sep 17 00:00:00 2001
From: forsyth2 <30700190+forsyth2@users.noreply.github.com>
Date: Tue, 14 Jan 2025 12:41:26 -0800
Subject: [PATCH] Update PR template (#12)

---
 .github/pull_request_template.md | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)

diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
index 2bc3200..00d4be7 100644
--- a/.github/pull_request_template.md
+++ b/.github/pull_request_template.md
@@ -4,9 +4,20 @@
 Select one: This pull request is...
 - [ ] a bug fix: increment the patch version
 - [ ] a small improvement: increment the minor version
+- [ ] a new feature: increment the minor version
 - [ ] an incompatible (non-backwards compatible) API change: increment the major version
 
-## 1. Does this do what we want it to do?
+Please fill out either the "Small Change" or "Big Change" section, and delete the other.
+
+## Small Change
+
+- [ ] To merge, I will use "Squash and merge". That is, this change should be a single commit.
+
+## 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
@@ -18,7 +29,7 @@ Required:
 - [ ] Product Management: I have confirmed with the stakeholders that the objectives above are correct and complete.
 - [ ] Testing: I have considered likely and/or severe edge cases and have included them in testing.
 
-## 2. Are the implementation details accurate & efficient?
+### 2. Are the implementation details accurate & efficient?
 
 Required:
 - [ ] Logic: I have visually inspected the entire pull request myself.
@@ -27,12 +38,12 @@ Required:
 If applicable:
 - [ ] Dependencies: This pull request introduces a new dependency. I have discussed this requirement with at least one other team member. The dependency is noted in `zppy-interfaces/conda`, not just an `import` statement.
 
-## 3. Is this well documented?
+### 3. Is this well documented?
 
 Required:
 - [ ] Documentation: by looking at the docs, a new user could easily understand the functionality introduced by this pull request.
 
-## 4. Is this code clean?
+### 4. Is this code clean?
 
 Required:
 - [ ] Readability: The code is as simple as possible and well-commented, such that a new team member could understand what's happening.