Conversation
pnpm/action-setup v6 no longer ships node-gyp on PATH, so node-pty's install hook (which falls back to `node-gyp rebuild` when no linux-x64 prebuild matches) fails. Install node-gyp globally before `pnpm install` in the test and release workflows so the bump to v6 (#390) can land.
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
Summary
npm install -g node-gypstep beforepnpm installintests.yml(both jobs) andrelease.yml.pnpm/action-setupv4 → v6 bump in chore(actions): bump pnpm/action-setup from 4 to 6.0.3 #390, which currently fails because v6 no longer ships node-gyp on PATH andnode-pty@1.1.0falls back tonode-gyp rebuild(no linux-x64 prebuild matches).Why
The failing log on #390 shows:
```
node-pty install: > Rebuilding because directory .../prebuilds/linux-x64 does not exist
node-pty install: sh: 1: node-gyp: not found
ELIFECYCLE Command failed.
```
Main is currently still on
pnpm/action-setup@v4(which bundles node-gyp), so this change is a no-op on main today and only takes effect once #390 lands. Including the same step inrelease.ymlso the release path doesn't break the moment #390 merges.