Skip to content

fix: enforce required changesets - #24

Merged
santi020k merged 2 commits into
mainfrom
feat/enforce-changesets
Aug 21, 2026
Merged

santi020k merged 2 commits into
mainfrom
feat/enforce-changesets

Conversation

@santi020k

Copy link
Copy Markdown
Owner

Summary

  • fail package pull requests when Changesets reports no pending release
  • automatically label Dependabot updates as changelog-free
  • preserve the explicit skip-changelog escape hatch for documentation and infrastructure work

Validation

  • pnpm run validate
  • actionlint
  • git diff --check

@santi020k santi020k added the skip-changelog Skip changeset requirement for release or non-user-facing changes label Aug 21, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 888992dfdf

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/dependabot.yml Outdated
directory: /
labels:
- dependencies
- skip-changelog

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Do not skip releases for runtime dependency updates

Because this label is configured for the entire npm ecosystem entry, Dependabot also applies it to production-dependency PRs, not only the grouped development updates. Those dependencies are bundled into the published CLI, so merging a runtime fix without a changeset leaves the package version unchanged; I also checked .github/workflows/release.yml, whose push paths exclude the lockfile and workspace catalog, meaning the merge does not trigger publishing and users may never receive the update until an unrelated release. Restrict this exemption to development-only updates or otherwise require release-producing npm updates to include a changeset.

AGENTS.md reference: AGENTS.md:L11-L11

Useful? React with 👍 / 👎.

Keep the automatic changelog exemption limited to GitHub Actions updates so runtime package changes cannot merge without a release.
@santi020k
santi020k merged commit d9b88e1 into main Aug 21, 2026
8 checks passed
@santi020k
santi020k deleted the feat/enforce-changesets branch August 21, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Skip changeset requirement for release or non-user-facing changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant