Fix rust coverage exit handling#52
Conversation
Reviewer's GuideThis change enhances the Cargo invocation helper to correctly propagate non-zero exit codes and cleans up the script by moving the read_previous helper into a more logical position. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
dd52380
into
codex/log-current-and-previous-coverage-percentage
* generate-coverage: log current and previous coverage * generate-coverage: share baseline reader * Refactor baseline reader * fix rust coverage failure handling (#52)
Summary
Testing
make lintmake testhttps://chatgpt.com/codex/tasks/task_e_688b5319574c8322a11b30739b5d4b31
Summary by Sourcery
Fix Rust coverage script to correctly propagate Cargo exit codes by extracting the read_previous helper out of the _run_cargo function.
Bug Fixes:
Enhancements: