Skip to content

#6306 Update PR template - #6309

Open
martgil wants to merge 1 commit into
masterfrom
issue-6306-improve-pr-template
Open

martgil wants to merge 1 commit into
masterfrom
issue-6306-improve-pr-template

Conversation

@martgil

@martgil martgil commented Oct 5, 2026

Copy link
Copy Markdown
Collaborator

This PR only updates the PR template, from which it adds extra steps for checking PR reviews.


Tests (delete all except exactly one):

  • Does not need tests (refactor only, docs or internal changes)
  • Difficult to test (explain why)
  • Not worth testing
  • Tests will be added later (issue #...)
  • Tests added or updated

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

@martgil
martgil requested a review from sosnovsky as a code owner October 5, 2026 10:05
@martgil

martgil commented Oct 5, 2026

Copy link
Copy Markdown
Collaborator Author

@sosnovsky - This one is ready for a review. Thanks!

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