GitHub: Add tests checkbox to PR template checklist
Adds an explicit reminder for both authors and reviewers to ensure tests are added and passing for new functionality and bug fixes. Also renames the section from "Special notes for your reviewer" to "Checklist" with shared accountability framing.
This commit is contained in:
@@ -42,9 +42,11 @@ Usage: "Fixes #<issue number>", or "Fixes (paste link of issue)"
|
||||
|
||||
Fixes #
|
||||
|
||||
**Special notes for your reviewer:**
|
||||
**Checklist:**
|
||||
|
||||
Both author and reviewer should ensure the following:
|
||||
|
||||
Please check that:
|
||||
- [ ] It works as expected from a user's perspective.
|
||||
- [ ] If this is a pre-GA feature, it is behind a feature toggle.
|
||||
- [ ] The docs are updated, and if this is a [notable improvement](https://grafana.com/docs/writers-toolkit/contribute/release-notes/#how-to-determine-if-content-belongs-in-whats-new), it's added to our [What's New](https://grafana.com/docs/writers-toolkit/contribute/release-notes/) doc.
|
||||
- [ ] Tests are added and passing, both for new functionality and for any bug fixes.
|
||||
|
||||
Reference in New Issue
Block a user