Repository navigation
fix(relay): follow usage-rs 6.11.1 moving env aliases under FlagMeta::extra - #4006
Conversation
…:extra usage-rs 6.11.1 (a patch release) moved FlagMeta's env_fallback and deprecated_env into FlagMeta::extra. release-plz's lockfile refresh in #3945 picked it up, so released_cli.rs stopped compiling and blocked the release. Take 6.11.1 on main and read the moved fields. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
MERGEPositive improvement? Yes. Worth the complexity? Yes. Two call-site field paths plus a Different approach? Pinning Note: This is an automated review, not the maintainer's decision |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
Included review availability: Your plan provides up to 4 included reviews per hour; 0 remain after this review. WalkthroughThe Merge Risk: ⚪ Minimal · up to No concrete regression remains in the reviewed changes, so no identified issue blocks merging. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches✨ Simplify code
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 |
Unblocks release-plz #3945. Its
CheckandTestjobs fail with:usage-rs6.11.1, a patch release from today, movedFlagMeta::env_fallback/deprecated_envintoFlagMeta::extra(FlagExtra). Thecargo updatein release-plz's lockfile refresh picked it up. Main still locks 6.11.0 and onlycheck-changedbuilds moq-relay's tests, so other PRs never hit it.This PR:
cargo update -p usage-rs, which moves allusage-*crates to 6.11.1);released_cli.rs.Only this test reads those fields.
cargo test --locked -p moq-relay --test released_clipasses (2/2).🤖 Generated with Claude Code
(written by Opus 5.5)