rust-lang/rust-clippy

Support for upcoming `param_attrs` feature

Open

#4.474 aberto em 30 de ago. de 2019

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Rust (1.391 forks)batch import
C-enhancementE-needs-testS-needs-discussiongood first issue

Métricas do repositório

Stars
 (10.406 stars)
Métricas de merge de PR
 (Mesclagem média 19d 22h) (113 fundiu PRs em 30d)

Description

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).

Guia do colaborador

Direção de pesquisa
Revise o PR do Rust vinculado (https://github.com/rust lang/rust/pull/64010) para entender o recurso param attrs. Em seguida, examine os lints existentes do clippy para identificar quais poderiam se beneficiar de atributos de parâmetro, como aqueles que verificam parâmetros não utilizados ou problemas de estilo. Crie uma lista dos lints afetados e atualize os para suportar a nova sintaxe de atributo, se necessário. Atualmente, não há comentários ou PRs que forneçam mais orientações.
Pilha de tecnologia
rust
Domain
developer experience
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Rust
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40