rust-lang/rust-clippy

Manual assert_eq! (enhance `if_then_panic`)

Open

#7,716 创建于 2021年9月24日

在 GitHub 查看
 (7 评论) (0 反应) (1 负责人)Rust (1,391 fork)batch import
C-enhancementgood first issue

仓库指标

Star
 (10,406 star)
PR 合并指标
 (平均合并 19天 22小时) (30 天内合并 113 个 PR)

描述

We just added if_then_panic, but then I realized there's a possible enhancement for assert_eq!/assert_ne!:

// before
if a != b {
    panic!("hi");
}
// after
assert_eq!(a, b, "hi");

I think a rename to manual_assert would be best. But we could also keep the current name (it still kinda makes sense) or create a new lint for this case.

贡献者指南

研究方向
探索 rust clippy 中现有的 `if then panic` lint 实现。该问题建议增强此 lint，使其也能在 `assert eq!`/`assert ne!` 模式上触发。主要决策是：将 lint 重命名为 `manual assert`、保留当前名称，还是创建一个新的 lint。查看评论中的讨论和 PR #7669 以了解背景。分配的用户是 Labelray，因此请检查他们的进展或评论。实现将涉及在 lint 逻辑中添加新的模式匹配。可参考 `clippy lints/src/if then panic.rs` 等文件。
技术栈
rust
领域
tooling
议题类型
功能
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
Rust
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
65