Skip to content

ci: require a package.json version bump on every PR#9

Merged
eitanp461 merged 1 commit into
masterfrom
ci/require-version-bump
Jun 7, 2026
Merged

ci: require a package.json version bump on every PR#9
eitanp461 merged 1 commit into
masterfrom
ci/require-version-bump

Conversation

@eitanp461

Copy link
Copy Markdown
Contributor

Summary

Every merge to master triggers npm publish (release.yml), so a PR that forgets to bump package.json version republishes the same version and breaks the n8n UI's update detection. This adds a fail-fast gate to the PR check so forgetting is impossible.

Changes

  • Add a Require version bump step to pr-check.yml that fails early (before npm ci) when the PR's package.json version matches origin/master
  • Bump version to 0.1.1 so this PR passes its own gate

Related

How to Test

  • This PR's own check run: the "Require version bump" step should pass with Version bump OK: 0.1.0 -> 0.1.1
  • Negative case: any PR leaving the version untouched will fail that step with an annotated error on package.json

Notes

🤖 Generated with Claude Code

Every merge to master triggers npm publish (release.yml), so a PR that
forgets to bump the version republishes the same version and breaks the
n8n UI update detection. Fail the PR check early when package.json
version matches master.

Bump to 0.1.1 so this PR passes its own gate.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@eitanp461 eitanp461 requested a review from sveta-slepner June 7, 2026 08:39
@eitanp461 eitanp461 merged commit 5347423 into master Jun 7, 2026
1 check passed
@eitanp461 eitanp461 deleted the ci/require-version-bump branch June 7, 2026 08:41
eitanp461 added a commit that referenced this pull request Jun 7, 2026
Every merge to master publishes; 0.1.1 is taken by PR #9.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.

2 participants