Conversation
Changesets action commits version bumps which triggers lefthook's pre-commit hook in CI. The hook runs nx affected which fails without an Nx Cloud DTE agent context. Setting LEFTHOOK=0 skips hooks for all git operations in the publish jobs. Also replaces stale HUSKY=0 with LEFTHOOK=0 in the snapshot job.
|
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 9 minutes and 11 seconds. ⌛ 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 (1)
✨ 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 |
|
View your CI Pipeline Execution ↗ for commit 6c8d395
☁️ Nx Cloud last updated this comment at |
Codecov Report✅ All modified and coverable lines are covered by tests. ❌ Your project status has failed because the head coverage (15.67%) is below the target coverage (40.00%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #570 +/- ##
===========================================
- Coverage 70.90% 15.67% -55.24%
===========================================
Files 53 154 +101
Lines 2021 26666 +24645
Branches 377 1127 +750
===========================================
+ Hits 1433 4180 +2747
- Misses 588 22486 +21898 🚀 New features to boost your workflow:
|
@forgerock/davinci-client
@forgerock/device-client
@forgerock/journey-client
@forgerock/oidc-client
@forgerock/protect
@forgerock/sdk-types
@forgerock/sdk-utilities
@forgerock/iframe-manager
@forgerock/sdk-logger
@forgerock/sdk-oidc
@forgerock/sdk-request-middleware
@forgerock/storage
commit: |
|
Deployed 1b565a9 to https://ForgeRock.github.io/ping-javascript-sdk/pr-570/1b565a9df89dfd6fff5a7c712771a6008bb7a695 branch gh-pages in ForgeRock/ping-javascript-sdk |
📦 Bundle Size Analysis📦 Bundle Size Analysis🚨 Significant Changes🔻 @forgerock/device-client - 0.0 KB (-9.7 KB, -100.0%) 📊 Minor Changes📈 @forgerock/device-client - 9.9 KB (+0.2 KB) ➖ No Changes➖ @forgerock/sdk-utilities - 11.2 KB 14 packages analyzed • Baseline from latest Legend🆕 New package ℹ️ How bundle sizes are calculated
🔄 Updated automatically on each push to this PR |
Summary
LEFTHOOK=0to thepublish-or-prjob so changesets' version commit doesn't trigger lefthook pre-commit hooks in CIHUSKY=0withLEFTHOOK=0in thesnapshotjobContext
Merging #564 triggered the publish workflow, where changesets ran
git committo version packages. This fired lefthook's pre-commit hook which runsnx affected— but there's no Nx Cloud DTE agent context for that secondary commit, causing CI to fail.Fixes: https://github.com/ForgeRock/ping-javascript-sdk/actions/runs/24483747421/job/71553933332
Test plan