rust-lang/rust-clippy

Manual assert_eq! (enhance `if_then_panic`)

Open

#7 716 ouverte le 24 sept. 2021

Voir sur GitHub
 (7 commentaires) (0 réactions) (1 assigné)Rust (1 391 forks)batch import
C-enhancementgood first issue

Métriques du dépôt

Stars
 (10 406 stars)
Métriques de merge PR
 (Merge moyen 19j 22h) (113 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Explorez l'implémentation existante du lint `if then panic` dans rust clippy. L'issue suggère de l'améliorer pour qu'il se déclenche également sur les motifs `assert eq!`/`assert ne!`. La décision principale est de savoir s'il faut renommer le lint en `manual assert`, conserver le nom actuel ou créer un nouveau lint. Consultez la discussion dans les commentaires et la PR #7669 pour le contexte. L'utilisateur assigné est Labelray, donc vérifiez ses progrès ou commentaires. L'implémentation impliquerait d'ajouter une nouvelle correspondance de motif dans la logique du lint. Voir des fichiers comme `clippy lints/src/if then panic.rs` pour référence.
Stack technique
rust
Domaine
tooling
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65