rust-lang/rust-clippy

Manual assert_eq! (enhance `if_then_panic`)

Open

#7.716 aberto em 24 de set. de 2021

Ver no GitHub
 (7 comments) (0 reactions) (1 assignee)Rust (1.391 forks)batch import
C-enhancementgood 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

We just added if_then_panic, but then I realized there's a possible enhancement for assert_eq!/assert_ne!:

// before
if a != b {
    panic!("hi");
}
// after
assert_eq!(a, b, "hi");

I think a rename to manual_assert would be best. But we could also keep the current name (it still kinda makes sense) or create a new lint for this case.

Guia do colaborador

Direção de pesquisa
Explore a implementação existente do lint `if then panic` no rust clippy. A issue sugere aprimorá lo para também acionar em padrões `assert eq!`/`assert ne!`. A principal decisão é se deve renomear o lint para `manual assert`, manter o nome atual ou criar um novo lint. Veja a discussão nos comentários e a PR #7669 para contexto. O usuário atribuído é Labelray, então verifique o progresso ou comentários dele. A implementação envolveria adicionar uma nova correspondência de padrão na lógica do lint. Consulte arquivos como `clippy lints/src/if then panic.rs` para referência.
Pilha de tecnologia
rust
Domain
tooling
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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.
65