Repository navigation
feat!: run license-checker-rseidelsohn in check-licenses - #130
Merged
Merged
Conversation
Settle on license-checker-rseidelsohn 5.x (final stable) instead of the 0.x @lizenz/checker. Its bin is `license-checker-rseidelsohn`, not `license-checker`, so check-licenses now invokes that name. BREAKING CHANGE: sites must replace their `@lizenz/checker` (or `license-checker`) devDependency with `license-checker-rseidelsohn`. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
taraxvoid
added a commit
that referenced
this pull request
Oct 12, 2026
…ckout (#141) Why 0.12.2 never reached npm: 1. Run 38204136468 (push of #130, f84a0f6) was in flight when release PR #137 merged (56239c2). 2. release-please reads the branch through the API, so that run created v0.12.2 on 56239c2 and set `release_created=true`. 3. The publish job checked `git merge-base --is-ancestor v0.12.2 HEAD`, but HEAD was the run's own checkout (f84a0f6), so it failed with "Tag v0.12.2 is not on branch main". 4. #137's own run (38204156854) found v0.12.2 already released, so `release_created` was false and publish was skipped. Fix: compare the tag against `refs/remotes/origin/<branch>`, which the job's `fetch-depth: 0` checkout fetches after release-please has finished. This keeps the same guarantee (the tag is on this channel's branch) and works no matter which push's run creates the release. 0.12.3 replaced 0.12.2 on npm, so I'm not republishing 0.12.2. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Settle on license-checker-rseidelsohn 5.x (final stable) instead of the
0.x @lizenz/checker. Its bin is
license-checker-rseidelsohn, notlicense-checker, so check-licenses now invokes that name.BREAKING CHANGE: sites must replace their
@lizenz/checker(orlicense-checker) devDependency withlicense-checker-rseidelsohn.Co-Authored-By: Claude Opus 5.5 noreply@anthropic.com