Skip to content

fix(ci): satisfy clippy doc_lazy_continuation in rename.rs#64

Merged
vasie1337 merged 1 commit into
mainfrom
claude/repo-status-update-jtxcaz
Jun 10, 2026
Merged

fix(ci): satisfy clippy doc_lazy_continuation in rename.rs#64
vasie1337 merged 1 commit into
mainfrom
claude/repo-status-update-jtxcaz

Conversation

@vasie1337

Copy link
Copy Markdown
Member

Problem

CI has been red on main since 77e44ed / d9b84a1. The failure is not a logic bug — the cargo clippy --release --all-targets -- -D warnings step trips on a doc-comment lint, and because clippy runs first, the Build and Test steps are skipped entirely.

error: doc list item without indentation
  --> src/passes/rename.rs:802:5
  = note: `-D clippy::doc_lazy_continuation` implied by `-D warnings`

The trailing Returns None ... sentence directly followed a numbered list in the usage_hint doc comment, so clippy read it as a lazily-continued list item.

Fix

Insert a blank doc line so the sentence reads as its own paragraph. One-line, comment-only change — no behavior impact.

Verification

cargo clippy --release --all-targets -- -D warnings now passes locally (clean finish, no warnings).

https://claude.ai/code/session_01X2eNjTxiuWTZ713JVN9nbC


Generated by Claude Code

…uation

The trailing 'Returns None ...' sentence followed the numbered list
directly, tripping clippy::doc_lazy_continuation under -D warnings and
failing CI on main. Add a blank doc line so it reads as its own
paragraph.

https://claude.ai/code/session_01X2eNjTxiuWTZ713JVN9nbC
@vasie1337 vasie1337 merged commit 09b8308 into main Jun 10, 2026
1 check passed
@vasie1337 vasie1337 deleted the claude/repo-status-update-jtxcaz branch June 10, 2026 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants