rust-lang/rust-clippy

Support for upcoming `param_attrs` feature

Open

#4474 aperta il 30 ago 2019

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Rust (1391 fork)batch import
C-enhancementE-needs-testS-needs-discussiongood first issue

Metriche repository

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

Descrizione

param_attrs is likely to be stabilized in Rust 1.39

The param_attrs feature is going to allow lint attributes on function, method and closure parameters:

fn foo(
    a: i32,
    #[allow(unused_variables)] b: i32,
) {}

We should review how this feature ties in with our existing lints. Apparently it did not cause any toolstate breakage when it was merged.

Specifically, we should create a list of lints that could benefit from parameter attributes and then update those lints to make use of them (if that's even required).

Guida contributor

Direzione di ricerca
Rivedi la PR di Rust collegata (https://github.com/rust lang/rust/pull/64010) per comprendere la funzionalità param attrs. Quindi, esamina i lint esistenti di clippy per identificare quali potrebbero beneficiare degli attributi dei parametri, come quelli che controllano i parametri inutilizzati o problemi di stile. Crea un elenco dei lint interessati e aggiornali per supportare la nuova sintassi degli attributi, se necessario. Al momento non ci sono commenti o PR che forniscono ulteriori indicazioni.
Tech stack
rust
Dominio
developer experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40