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