fix(node-ci): set up Node for bun projects too - #40
Conversation
setup-node was gated `if: package-manager != 'bun'`, on the assumption bun projects don't need Node. But native and postinstall deps (sharp, cypress, …) spawn `node` during `bun install`. On GitHub-hosted runners the system Node is modern so it worked by accident; on a self-hosted runner with an old system Node, `bun install` crashes in the postinstall (e.g. cypress's cli parsed under Node 12 → "Unexpected token ="). Runs setup-node for every package manager now. bun still gets setup-bun; it just also gets a pinned modern Node on PATH. No npm/yarn/pnpm cache is enabled for bun (it manages its own store), so nothing else changes for bun callers, and non-bun paths are byte-identical.
|
Warning Review limit reached
Next review available in: 26 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
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 |
|
Caution The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased. |
1 participant
setup-nodewas gatedif: package-manager != 'bun'. But native/postinstall deps (sharp, cypress) spawnnodeduringbun install. On GitHub-hosted runners system Node is modern so it worked by accident; on a self-hosted runner with old system Node,bun installcrashes in cypress's postinstall (parsed under Node 12). Runs setup-node for every PM now — bun also gets a pinned modern Node on PATH, no npm cache for bun, non-bun paths unchanged.