fix(cardwire-cli): label the battery auto switch mode output correctly - #265
plastininikolay wants to merge 1 commit into
Conversation
`cardwire config battery-auto-switch-mode` printed "BatteryAutoSwitch:", the same label as `cardwire config battery-auto-switch`. Print "BatteryAutoSwitchMode:" instead. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
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: Repository UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe CLI output label for ChangesCLI output
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Change: Bug fix · Severity of issue fixed: Low Suggested reviewers: Merge Risk: ⚪ Minimal · up to The CLI now labels this setting as a mode rather than a switch. No actionable merge-blocking risk remains. 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
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 |
|
Why did you need Claude opus 5.5 assistance to edit a simple string |
|
I noticed the wrong label myself while setting up Cardwire. Claude only helped with the routine part: the PR description, the checklist and running fmt/clippy. I mention AI help on every contribution, even small ones like this, since many projects ask for it and I'd rather be upfront. |
Description
cardwire config battery-auto-switch-modeprinted its value with the label of the neighbouring setting:It now prints
BatteryAutoSwitchMode: Hybrid.Assisted-by: Claude Opus 5.5
Checklist:
cargo clippy -p cardwire-cli -- -D warningsis clean