Skip to content

fix: clippy lints introduced by Rust 1.99 - #36

Open
heraldb wants to merge 1 commit into
naust-mail:mainfrom
heraldb:fix/clippy-rust-1.99
Open

heraldb wants to merge 1 commit into
naust-mail:mainfrom
heraldb:fix/clippy-rust-1.99

Conversation

@heraldb

@heraldb heraldb commented Oct 29, 2026

Copy link
Copy Markdown

Problem

cargo clippy -- -D warnings, as run in CI, fails on main with the current
stable toolchain (Rust 1.99):

  • clippy::double_must_use on every #[async_trait] trait method
    (ImapClient, SmtpClient). The lint fires on code generated by
    async-trait 0.1.89.
  • clippy::map_or_identity in smtp::client::filter_envelope_headers.

Change

  • Bump async-trait 0.1.89 → 0.1.92 (Cargo.toml and Cargo.lock), which
    generates lint-clean code. This adds syn 3 as a build-time dependency; no
    other packages change.
  • .map_or((raw, ""), |p| p) → .unwrap_or((raw, "")).

No behaviour change. cargo clippy --all-targets -- -D warnings is clean,
and cargo test passes.

🤖 Generated with Claude Code

`cargo clippy -- -D warnings` (as run in CI) fails on the current stable
toolchain:

- clippy::double_must_use on every #[async_trait] trait method. Fixed in
  async-trait 0.1.92; bump the requirement and lockfile (adds syn 3 as a
  build dependency).
- clippy::map_or_identity in filter_envelope_headers: use unwrap_or.

No behaviour change.

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.

1 participant