Fix: pluralize doctor working-tree file count correctly - #457
Conversation
`fledge doctor` printed "uncommitted changes (1 files)". Use the singular "file" when the count is 1 and "files" otherwise. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016AvsKakjAc3EKN2ztcMfYi
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the git check in src/doctor.rs to correctly pluralize the word 'file' based on the number of uncommitted changes. There are no review comments, and I have no additional feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
0xGaspar
left a comment
0xGaspar
left a comment
There was a problem hiding this comment.
Correct singular/plural fix. count == 1 guard is right and the inline-captured format args are tidy. LGTM.
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
✅ Corvin says...
_
<(^\ .oO(Caw! ^v^)
|/(\
\(\\
" "\\
"Caw! Found a shiny new spec!"
CI Summary
| Check | Status |
|---|---|
| Dependency Audit | ✅ Passed |
| Integration (3 OS) | ✅ Passed |
| Lint (fmt + clippy) | ✅ Passed |
| Spec Validation | ✅ Passed |
| Tests (3 OS) | ✅ Passed |
Powered by corvid-pet
2 participants
Summary
fledge doctorpreviously always printed "uncommitted changes (N files)", so a single change read as "(1 files)".Test Plan
cargo fmtcargo buildcargo clippy --bin fledge -- -D warnings(clean)cargo test --bin fledge doctor(9 passed)🤖 Generated with Claude Code
Closes #453
https://claude.ai/code/session_016AvsKakjAc3EKN2ztcMfYi