Skip to content

ci: enforce changesets on pull requests#610

Merged
brendanjryan merged 1 commit into
mainfrom
brendanjryan/enforce-changesets-ci
Jul 1, 2026
Merged

ci: enforce changesets on pull requests#610
brendanjryan merged 1 commit into
mainfrom
brendanjryan/enforce-changesets-ci

Conversation

@brendanjryan

@brendanjryan brendanjryan commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Motivation

Package-impacting pull requests should fail normal CI when they omit a Changeset, while generated release pull requests and intentionally unreleased changes need an explicit escape hatch.

Summary

  • Added a Changesets check to the existing Verify / Checks CI job.
  • Used the PR merge commit parent (HEAD^1) as the comparison point with a shallow depth of 2.
  • Skipped the check for generated changeset-release/main release pull requests titled chore: version packages.
  • Added support for a skip-changeset label to bypass the check, with PR CI rerunning when labels change.

Key design considerations

  • Kept the existing main release workflow unchanged.
  • Avoided a full-history checkout and separate PR job by reusing normal CI.

@pkg-pr-new

pkg-pr-new Bot commented Jul 1, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/mppx@610

commit: d413f7a

@brendanjryan brendanjryan marked this pull request as ready for review July 1, 2026 22:06
@brendanjryan brendanjryan force-pushed the brendanjryan/enforce-changesets-ci branch from ba35d52 to 268d287 Compare July 1, 2026 22:08
@brendanjryan brendanjryan force-pushed the brendanjryan/enforce-changesets-ci branch from 268d287 to d413f7a Compare July 1, 2026 22:17
@brendanjryan brendanjryan added the skip-changeset Skip the Changesets PR check label Jul 1, 2026
@brendanjryan brendanjryan merged commit 6419c44 into main Jul 1, 2026
23 of 25 checks passed
@brendanjryan brendanjryan deleted the brendanjryan/enforce-changesets-ci branch July 1, 2026 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changeset Skip the Changesets PR check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant