Repository navigation
release: set up Node 24 after Bun install - #5
Conversation
|
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 54 minutes. View limit details
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Bun and publishing setup order .github/workflows/release.yml |
The workflow runs Setup Bun and install before Setup Node + npm for trusted publishing. Updated comments explain why Node 24’s npm 11 is configured last and state that switching Node has no ABI consequences in this repository. |
Priority:
Estimated code review effort: 2 (Simple) | ~8 minutes
Change: Bug fix
Merge Risk: 
Minimal · up to d3990
-
The publishing workflow order is correct and no unresolved merge risk was identified.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
The composite setup action runs its own setup-node (Node 22, npm 10.x), which put npm 10 back on PATH after our Node 24 step. npm 10 cannot do OIDC, so the first trusted publish went out anonymous and the registry 404d every PUT. Moving our step last guarantees npm 11 at publish time. Retries the unpublished 0.1.1s on merge.
Summary by CodeRabbit