Skip to content

chore(lint): enable clippy::explicit_into_iter_loop - #248

Open
tupe12334 wants to merge 1 commit into
mainfrom
clippy/enable-explicit_into_iter_loop
Open

tupe12334 wants to merge 1 commit into
mainfrom
clippy/enable-explicit_into_iter_loop

Conversation

@tupe12334

Copy link
Copy Markdown
Member

Closes #247

What

Promote one more Clippy lint in the incremental ratchet: clippy::explicit_into_iter_loop = "deny" in the [lints.clippy] table of core/Cargo.toml.

clippy::explicit_into_iter_loop flags for x in items.into_iter() and asks for for x in items. A for loop already calls IntoIterator::into_iter, so the explicit call is redundant.

The lint pairs with clippy::explicit_iter_loop, which the crate already denies, so all for loops now follow one idiom.

Changes

  • core/Cargo.toml: add explicit_into_iter_loop = "deny" next to explicit_iter_loop.
  • No code changes: the lint surfaced zero violations.

Verification

Run in core/:

  • cargo fmt --check — clean
  • cargo clippy --all-targets --all-features -- -D warnings — no issues
  • cargo test — 191 passed, 0 failed

This pull request was opened automatically by the moadim routine "Clippy lint improvements → issue + PR (Rust repos) → Slack".

🤖 Generated with Claude Code

Closes #247

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.

Enable clippy::explicit_into_iter_loop lint

1 participant