Conversation
The sergeysova/jq-action@v2 uses a Docker container that only provides jq within the container context. Once the container exits, jq is no longer available in PATH for subsequent test steps. This caused auto-updater integration tests to fail with exit code 127 (command not found) when trying to use jq in update.sh and check.sh scripts. Root cause analysis: - jq-action runs in a Docker container - After container exits, jq is not in PATH - BATS tests run in separate process without jq - Scripts using jq fail with "command not found" Solution: Install jq directly via apt-get so it persists in PATH for all test steps. Also removes unsupported 'cache: false' parameter from bats-action@3.0.1 which was causing warnings. Co-Authored-By: Claude <noreply@anthropic.com>
…workflow The update-checker.sh file was removed in commit #310 (feat(auto-updater): add config-based multi-marketplace support), replaced by update.sh. The backup/restore logic in release.yml was not updated and was causing failures attempting to backup a non-existent file. Root cause: release.yml:83-97 referenced update-checker.sh which no longer exists after commit 19583ac. Fix: Remove the obsolete backup/restore logic for update-checker.sh. Co-Authored-By: Claude <noreply@anthropic.com>
|
Warning Rate limit exceeded
⌛ 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. ✨ 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 |
1 participant
Summary
update-checker.shfileRoot Cause
Commit #310 (feat: auto-updater config-based multi-marketplace support) removed
update-checker.shand replaced it withupdate.sh. The release workflow'sbackup/restore logic was not updated and continued to reference the removed file.
Test plan
update-checker.shwas removed in commit 19583acupdate.shis the current file in that locationbats tests/- all 165 tests pass🤖 Generated with Claude Code