rust-lang/rust-clippy

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

Open

#14.253 geöffnet am 19. Feb. 2025

Auf GitHub ansehen
 (17 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Rust (1.391 Forks)batch import
E-help-wantedgood first issue

Repository-Metriken

Stars
 (10.406 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 16T 6h) (79 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Überprüfe zunächst die aktuellen Lints in `clippy lints/src/strings.rs` und identifiziere, welche (außer `string add`, `string add assign`, `string slice`) in das `methods` Modul verschoben werden können. Verstehe dann das im `methods` Modul verwendete Baumtraversierungsmuster, um die verschobenen Lints zu integrieren. Stelle abschließend sicher, dass keine doppelten Funktionen vorhanden sind, und aktualisiere die Testerwartungen entsprechend. Das Issue hat 16 Kommentare; überprüfe diese auf Anleitungen der Maintainer und potenzielle Hindernisse.
Tech Stack
rust
Domain
cli
Issue Type
Refactoring
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
RustGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70