Skip to content

Update PR template #13302

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

Jump to bottom
Merged
merged 2 commits into from
Oct 27, 2025
Merged

Update PR template #13302

merged 2 commits into from
Oct 27, 2025

Conversation

@viren-nadkarni
Copy link
Member

@viren-nadkarni viren-nadkarni commented Oct 23, 2025

Changes

This PR proposes following changes to our GitHub PR template:

  • Lines are broken to fit GitHub's narrow text box
  • Guidelines are moved into the sections they relate to
  • The optional Todo section is removed because it's rarely used
  • A new related section is added to link issues and related pages. This will systematise Linear IDs.

Related

Related to PNX-388

@viren-nadkarni viren-nadkarni self-assigned this Oct 23, 2025
@viren-nadkarni viren-nadkarni added semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Oct 23, 2025
@viren-nadkarni viren-nadkarni marked this pull request as ready for review October 23, 2025 11:47
silv-io
silv-io reviewed Oct 23, 2025
View reviewed changes
@viren-nadkarni @silv-io
Mention Linear IDs to prompt its use
01746f6 
Co-authored-by: Silvio Vasiljevic <silvio.vasiljevic@gmail.com>
@viren-nadkarni viren-nadkarni merged commit 612f68a into main Oct 27, 2025
6 checks passed
@viren-nadkarni viren-nadkarni deleted the issue-template branch October 27, 2025 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@silv-io silv-io silv-io left review comments

@bentsku bentsku bentsku left review comments

@k-a-il k-a-il k-a-il approved these changes

@alexrashed alexrashed Awaiting requested review from alexrashed alexrashed is a code owner

@dfangl dfangl Awaiting requested review from dfangl dfangl is a code owner

@dominikschubert dominikschubert Awaiting requested review from dominikschubert dominikschubert is a code owner

Assignees

@viren-nadkarni viren-nadkarni

Labels

docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@viren-nadkarni @silv-io @bentsku @k-a-il