Skip to content

chore: update PR template - #364

Merged
pyrooka merged 2 commits into
mainfrom
nb/update-pr-template
Sep 24, 2026
Merged

pyrooka merged 2 commits into
mainfrom
nb/update-pr-template

Conversation

@pyrooka

@pyrooka pyrooka commented Sep 24, 2026

Copy link
Copy Markdown
Member

PR summary

Adds a checklist item and a dedicated section to the PR template asking contributors to provide evidence of a clean integration/examples test run when those tests are modified.

PR Checklist

Current vs new behavior

Previously, the PR template had no mention of integration/examples test evidence, so reviewers had to ask for it manually during review.

The template now includes:

  • A checklist item reminding contributors to run integration/examples tests (if modified) and include a summary below.
  • A new Integration / examples test evidence section where contributors paste a test run summary (e.g. final pass/fail counts), or write "N/A" if no integration tests or examples were changed.

Does this PR introduce a breaking change?

  • Yes
  • No

Integration / examples test evidence

N/A

Signed-off-by: Norbert Biczo <pyrooka@users.noreply.github.com>
Signed-off-by: Norbert Biczo <pyrooka@users.noreply.github.com>
@pyrooka
pyrooka merged commit 25c09b5 into main Sep 24, 2026
7 checks passed
@pyrooka
pyrooka deleted the nb/update-pr-template branch September 24, 2026 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant