Skip to content

Commit de12435

Browse files
authored
Merge pull request #3308 from github/mbg/pr-template/nov25
Add additional options to PR template and clarify some
2 parents ffa63f0 + 7bcdb4b commit de12435

File tree

1 file changed

+27
-7
lines changed

1 file changed

+27
-7
lines changed

.github/pull_request_template.md

Lines changed: 27 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,14 +18,25 @@ For internal use only. Please select the risk level of this change:
1818

1919
#### Which use cases does this change impact?
2020

21-
<!-- Delete options that don't apply. -->
21+
<!-- Delete options that don't apply. If in doubt, do not delete an option. -->
22+
23+
Workflow types:
24+
25+
- **Advanced setup** - Impacts users who have custom CodeQL workflows.
26+
- **Managed** - Impacts users with `dynamic` workflows (Default Setup, CCR, ...).
27+
28+
Products:
29+
30+
- **Code Scanning** - The changes impact analyses when `analysis-kinds: code-scanning`.
31+
- **Code Quality** - The changes impact analyses when `analysis-kinds: code-quality`.
32+
- **CCR** - The changes impact analyses for Copilot Code Reviews.
33+
- **Third-party analyses** - The changes affect the `upload-sarif` action.
34+
35+
Environments:
2236

23-
- **Advanced setup** - Impacts users who have custom workflows.
24-
- **Default setup** - Impacts users who use default setup.
25-
- **Code Scanning** - Impacts Code Scanning (i.e. `analysis-kinds: code-scanning`).
26-
- **Code Quality** - Impacts Code Quality (i.e. `analysis-kinds: code-quality`).
27-
- **Third-party analyses** - Impacts third-party analyses (i.e. `upload-sarif`).
28-
- **GHES** - Impacts GitHub Enterprise Server.
37+
- **Dotcom** - Impacts CodeQL workflows on `github.com`.
38+
- **GHES** - Impacts CodeQL workflows on GitHub Enterprise Server.
39+
- **Testing/None** - This change does not impact any CodeQL workflows in production.
2940

3041
#### How did/will you validate this change?
3142

@@ -54,6 +65,15 @@ For internal use only. Please select the risk level of this change:
5465
- **Alerts** - New or existing monitors will trip if something goes wrong with this change.
5566
- **Other** - Please provide details.
5667

68+
#### Are there any special considerations for merging or releasing this change?
69+
70+
<!--
71+
Consider whether this change depends on a different change in another repository that should be released first.
72+
-->
73+
74+
- **No special considerations** - This change can be merged at any time.
75+
- **Special considerations** - This change should only be merged once certain preconditions are met. Please provide details of those or link to this PR from an internal issue.
76+
5777
### Merge / deployment checklist
5878

5979
- Confirm this change is backwards compatible with existing workflows.

0 commit comments

Comments
 (0)