Skip to content

refactor(application-rs): 统一 workspace clippy 配置并修复告警 - #154

Merged
yansongda merged 8 commits into
mainfrom
refactor/application-rs-clippy
Aug 9, 2026
Merged

refactor(application-rs): 统一 workspace clippy 配置并修复告警#154
yansongda merged 8 commits into
mainfrom
refactor/application-rs-clippy

Conversation

@yansongda

@yansongda yansongda commented Aug 9, 2026

Copy link
Copy Markdown
Owner

变更内容

  • application-rs/Cargo.toml 中统一配置 workspace 级 clippy lint(参考 message 项目)。
  • 所有 crate 通过 [lints] workspace = true 继承配置。
  • 所有 #[cfg(test)] mod tests 模块统一添加 #![allow(clippy::all)],忽略单元测试的 clippy 检查。
  • 修复启用 lint 后暴露的非测试代码告警:
    • application-kernel: 合并重复 match arm、使用 i64::from、为启动期 expect 添加 allow
    • application-util: redundant closure/else、Duration::from_mins、启动期 expect allow
    • application-database: 显式 Platform::Unsupported、implicit clone、启动期 expect allow
    • application-api: let...else、implicit clone、redundant closure、公开子模块避免 unreachable_pub、补充 Panics 文档

验证

cargo clippy -- -D warnings
cargo fmt --all -- --check
cargo check --all-features

三条命令均已通过。

@yansongda
yansongda merged commit 0353fb2 into main Aug 9, 2026
5 checks passed
@yansongda
yansongda deleted the refactor/application-rs-clippy branch August 9, 2026 15:59
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