📝 CodeRabbit Chat: Implement requested code changes - #137
📝 CodeRabbit Chat: Implement requested code changes#137coderabbitai[bot] wants to merge 2 commits into
Conversation
regenerate_lockfiles aborted on the first cargo failure, leaving the workspace inconsistent with no inventory of which lockfiles were refreshed: manifests carry the new version while later lockfiles were never attempted. Attempt every manifest, collect the failures, and raise one aggregated LockfileRegenerationError after the loop. The message states that the workspace is inconsistent and lists each failed manifest with its repair command (cargo update --workspace --manifest-path ...). The exception chains from the first underlying failure so diagnostics such as a missing cargo executable survive aggregation. Manifest-path validation errors still raise immediately, as those are configuration mistakes rather than partial-update hazards. Tests cover the mid-list failure (remaining manifests still attempted; successes excluded from the repair list), multi-failure aggregation with exactly one repair command per failed manifest, and a syrupy snapshot of the aggregated message. Closes #84
|
Important Review skippedThis PR was authored by the user configured for CodeRabbit reviews. CodeRabbit does not review PRs authored by this user. It's recommended to use a dedicated user account to post CodeRabbit review feedback. ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Sorry @coderabbitai[bot], you have reached your weekly rate limit of 500000 diff characters.
Please try again later or upgrade to continue using Sourcery
![codescene-delta-analysis[bot]](https://avatars.githubusercontent.com/in/53921?s=60&v=4){kind=small}
9356db7 to
9aa5c72
Compare
03bf3d9 to
3c4d65a
Compare
1 participant
Code changes was requested by @leynos.
The following files were modified:
lading/commands/bump_lockfiles.py