rust-lang/rust-clippy

Refactor the `clippy_lints/src/strings.rs` file

Ouverte

#14 253 ouverte le 19 févr. 2025

Voir sur GitHubGuide contributeur
 (17 commentaires) (0 réaction) (1 personne assignée)Rust (1 391 forks)batch import
E-help-wantedgood first issue

Métriques du dépôt

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

Description

Description

With the exception of string_add, string_add_assign and string_slice (which share an implementation), all lints in strings.rs could be moved into the methods module in order to share the tree traversal with other method call and function call lints.

Guide contributeur

Direction de recherche
Tout d'abord, examinez les lints actuels dans `clippy lints/src/strings.rs` et identifiez lesquels (à l'exclusion de `string add`, `string add assign`, `string slice`) peuvent être déplacés vers le module `methods`. Ensuite, comprenez le modèle de parcours d'arbre utilisé dans le module `methods` pour intégrer les lints déplacés. Enfin, assurez vous qu'il n'y a pas de fonctionnalité en double et mettez à jour les attentes des tests en conséquence. L'issue comporte 16 commentaires ; vérifiez les pour obtenir des conseils des mainteneurs et des obstacles potentiels.
Stack technique
rust
Domaine
cli
Type d'issue
Refactorisation
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
RustGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70