rust-lang/rust-clippy

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

Aperta

#14.253 aperta il 19 feb 2025

Vedi su GitHubGuida contributor
 (17 commenti) (0 reazioni) (1 assegnatario)Rust (1391 fork)batch import
E-help-wantedgood first issue

Metriche repository

Star
 (10.406 stelle)
Metriche merge PR
 (Merge medio 19g 22h) (113 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Innanzitutto, rivedi i lint attuali in `clippy lints/src/strings.rs` e identifica quali (escludendo `string add`, `string add assign`, `string slice`) possono essere spostati nel modulo `methods`. Quindi, comprendi il pattern di attraversamento dell'albero utilizzato nel modulo `methods` per integrare i lint spostati. Infine, assicurati che non ci siano funzionalità duplicate e aggiorna di conseguenza le aspettative dei test. L'issue ha 16 commenti; controllali per eventuali indicazioni dei maintainer e potenziali blocchi.
Tech stack
rust
Dominio
cli
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70