rust-lang/rust-clippy

Manual assert_eq! (enhance `if_then_panic`)

Open

#7.716 geöffnet am 24. Sept. 2021

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Rust (1.391 Forks)batch import
C-enhancementgood first issue

Repository-Metriken

Stars
 (10.406 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 19T 22h) (113 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche die bestehende `if then panic` Lint Implementierung in rust clippy. Das Issue schlägt vor, sie zu erweitern, damit sie auch bei `assert eq!`/`assert ne!` Mustern ausgelöst wird. Die Hauptentscheidung ist, ob der Lint in `manual assert` umbenannt, der aktuelle Name beibehalten oder ein neuer Lint erstellt werden soll. Siehe die Diskussion in den Kommentaren und den PR #7669 für den Kontext. Der zugewiesene Benutzer ist Labelray, also überprüfe seinen Fortschritt oder Kommentare. Die Implementierung würde das Hinzufügen eines neuen Musterabgleichs in der Lint Logik beinhalten. Siehe Dateien wie `clippy lints/src/if then panic.rs` als Referenz.
Tech Stack
rust
Domain
tooling
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Rust
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65