[Epic][Rust] Unify workspace lint standards and enforce full Clippy warnings
- Dominant language
- Rust
- Stars
- 1.8k
- Forks
- 241
- Avg merge
- 2d 14h
- Merged PRs (30d)
- 22
Description
## Overview
This epic aims to unify the Auron project's Rust code quality standards by standardizing workspace-level Clippy lint configurations. Building on the already enforced `clippy::unwrap_used` safety rules, it will enable full `-D warnings` enforcement and batch clean up all temporarily allowed non-critical lints in `[workspace.lints.rust]` and `[workspace.lints.clippy]`, establishing a consistent and robust code quality baseline for all Rust crates.
## Subtasks
- [x] #1911
- [x] #1912
Rules: `unused_variables, dead_code, unused_imports, unused_must_use, deprecated`
- [ ] #1913
Rules: `unnecessary_unwrap, needless_question_mark, unnecessary_literal_unwrap, bind_instead_of_map, expect_fun_call, io_other_error, unnecessary_fallible_conversions`
- [ ] #1914
Rules: `missing_transmute_annotations, declare_interior_mutable_const, borrow_interior_mutable_const, uninit_vec, macro_metavars_in_unsafe, ptr_arg, borrowed_box`
- [ ] #1915
Rules: `manual_flatten, iter_cloned_collect, into_iter_on_ref, box_collection, useless_vec, len_without_is_empty, len_zero, get_first, mem_replace_option_with_none`
- [ ] #1916
Rules: `needless_range_loop, while_let_loop, while_let_on_iterator, explicit_counter_loop`
- [x] #1917
Rules: `useless_format, uninlined_format_args, to_string_in_format_args`
- [ ] #1918
Rules: `needless_borrow, needless_return, redundant_closure, redundant_locals, redundant_pattern_matching`
- [ ] #1919
Rules: `unnecessary_cast, unnecessary_to_owned, useless_asref, clone_on_copy, unit_arg, manual_repeat_n, manual_div_ceil`
- [x] #1920
Rules: `collapsible_if, collapsible_else_if, if_same_then_else, match_like_matches_macro, explicit_auto_deref, bool_assert_comparison`
- [ ] #1921
Rules: `upper_case_acronyms, module_inception, too_many_arguments, should_implement_trait, extra_unused_lifetimes, crate_in_macro_def, int_plus_one, derived_hash_with_manual_eq, approx_constant, op_ref`
Contributor guide
Research direction
Start with the workspace-level [workspace.lints.rust] and [workspace.lints.clippy] configurations and review the unchecked subtasks, beginning with the listed Clippy rules. Run the project's Clippy checks across all Rust crates and consider the work done when full -D warnings enforcement passes without the temporary allowances covered by subtasks #1913–#1916, #1918, #1919, and #1921.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100