Skip to content

refactor: enable extended clippy lint set#675

Merged
farazdagi merged 20 commits intomainfrom
refactor/clippy-lints
Apr 21, 2026
Merged

refactor: enable extended clippy lint set#675
farazdagi merged 20 commits intomainfrom
refactor/clippy-lints

Conversation

@farazdagi
Copy link
Copy Markdown
Contributor

@farazdagi farazdagi commented Apr 20, 2026

Summary

  • Enable additional clippy lints workspace-wide (correctness, style, and clarity categories)
  • Fix all existing violations across the codebase, one lint per commit for easy review

Lints enabled

# Correctness & safety
await_holding_lock = "deny"
unused_async = "deny"
large_futures = "deny"

# Style & clarity
cloned_instead_of_copied = "warn"
copy_iterator = "warn"
dbg_macro = "warn"
enum_glob_use = "warn"
explicit_into_iter_loop = "warn"
explicit_iter_loop = "warn"
flat_map_option = "warn"
implicit_clone = "warn"
map_flatten = "warn"
manual_let_else = "warn"
manual_ok_or = "warn"
manual_async_fn = "warn"
map_unwrap_or = "warn"
match_same_arms = "warn"
redundant_closure = "warn"
redundant_closure_call = "warn"
redundant_closure_for_method_calls = "warn"
redundant_else = "warn"
redundant_clone = "warn"
redundant_field_names = "warn"
redundant_test_prefix = "warn"
semicolon_if_nothing_returned = "warn"
todo = "warn"
uninlined_format_args = "warn"
unnested_or_patterns = "warn"

@farazdagi farazdagi self-assigned this Apr 20, 2026
@farazdagi farazdagi force-pushed the refactor/clippy-lints branch from 09777ae to ae8e81d Compare April 21, 2026 08:23
@coveralls
Copy link
Copy Markdown

coveralls commented Apr 21, 2026

Coverage Status

Coverage is 78.38%refactor/clippy-lints into main. No base build found for main.

@farazdagi farazdagi marked this pull request as ready for review April 21, 2026 08:40
@farazdagi farazdagi requested a review from a team as a code owner April 21, 2026 08:40
@farazdagi farazdagi enabled auto-merge (squash) April 21, 2026 08:44
@farazdagi farazdagi merged commit 6bfabe4 into main Apr 21, 2026
13 checks passed
@farazdagi farazdagi deleted the refactor/clippy-lints branch April 21, 2026 08:47
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.

3 participants