Skip to content

chore: require PRs to link an issue or discussion#2093

Merged
KRRT7 merged 1 commit intomainfrom
chore/require-linked-issue-on-prs
Apr 23, 2026
Merged

chore: require PRs to link an issue or discussion#2093
KRRT7 merged 1 commit intomainfrom
chore/require-linked-issue-on-prs

Conversation

@KRRT7
Copy link
Copy Markdown
Collaborator

@KRRT7 KRRT7 commented Apr 23, 2026

Closes #2079 (partial — infrastructure improvement)

Summary

  • Adds a PR template (.github/PULL_REQUEST_TEMPLATE.md) with a required "Linked issue or discussion" section
  • Adds a check-linked-issue CI job that validates the PR body contains an issue/discussion reference — blocks merge via the required-checks-passed gate
  • Updates CONTRIBUTING.md with the policy and motivation: have a conversation before the code so contributors get direction early and implementations fit the project's needs

Test plan

  • Open a test PR with no issue reference — CI should fail on check-linked-issue
  • Open a test PR with Closes #123 in the body — CI should pass
  • Verify bot PRs (dependabot) are skipped

- Add PR template with required linked issue/discussion section
- Add check-linked-issue CI job that validates PR body contains a
  reference (#123, Closes/Fixes/Relates, GitHub URL, or CF-# ticket)
- Wire into required-checks-passed gate so it blocks merge
- Update CONTRIBUTING.md with the policy and motivation
@github-actions github-actions Bot added the workflow-modified This PR modifies GitHub Actions workflows label Apr 23, 2026
@KRRT7 KRRT7 marked this pull request as ready for review April 23, 2026 08:07
@KRRT7 KRRT7 merged commit 14ca2c8 into main Apr 23, 2026
17 checks passed
@KRRT7 KRRT7 deleted the chore/require-linked-issue-on-prs branch April 23, 2026 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

workflow-modified This PR modifies GitHub Actions workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Achieve 100% unit test coverage

1 participant