Skip to content

Update PR template to prompt for test coverage #3210

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Copilot wants to merge 3 commits into from
Closed

Update PR template to prompt for test coverage #3210

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
1373f1f
Initial plan
Copilot Oct 17, 2025
80f8fb8
Update PR template to require test coverage or justification
Copilot Oct 17, 2025
fa79376
Make PR template more concise
Copilot Oct 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 7 additions & 3 deletions .github/pull_request_template.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,13 +29,16 @@ For internal use only. Please select the risk level of this change:

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

<!-- Delete options that don't apply. -->
<!-- Delete options that don't apply. Be explicit about test coverage. If no new tests are added, explain why they are not needed. -->

- **Test repository** - This change will be tested on a test repository before merging.
- **Unit tests** - I am depending on unit test coverage (i.e. tests in `.test.ts` files).
- **Unit tests** - I am depending on existing unit test coverage (i.e. tests in `.test.ts` files).
- **End-to-end tests** - I am depending on PR checks (i.e. tests in `pr-checks`).
- **New / updated tests** - I have added or updated unit / end-to-end tests to cover new or changed functionality (summarize below) OR I am providing a justification below for why test updates are not required.
- **Other** - Please provide details.
- **None** - I am not validating these changes.
- **None** - I am not validating these changes (provide justification below).

<!-- If you selected "New / updated tests" or omitted adding tests, briefly summarize test additions or rationale: -->

#### If something goes wrong after this change is released, what are the mitigation and rollback strategies?

Expand All @@ -57,5 +60,6 @@ For internal use only. Please select the risk level of this change:
### Merge / deployment checklist

- Confirm this change is backwards compatible with existing workflows.
- Tests (unit and/or end-to-end) have been added or updated as needed and are passing, OR a justification for not adding tests has been provided.
- Consider adding a [changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) entry for this change.
- Confirm the [readme](https://github.com/github/codeql-action/blob/main/README.md) and docs have been updated if necessary.
Loading