FuelLabs/fuels-rs

Add clippy check for overflowing arithmetic

Open

#1.238 aberto em 14 de dez. de 2023

Ver no GitHub
 (0 comments) (0 reactions) (1 assignee)Rust (1.360 forks)batch import
Securitygood first issuetech-debtv1.0

Métricas do repositório

Stars
 (43.211 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Our fuzzing efforts have mainly discovered unchecked arithmetic. We should add the following clippy check to our CI to disallow unchecked arithmetic in the future: https://rust-lang.github.io/rust-clippy/master/index.html#/arithmetic_side_effects

Guia do colaborador

Direção de pesquisa
A configuração do CI provavelmente está em um arquivo de fluxo de trabalho do GitHub Actions (por exemplo, .github/workflows/ci.yml). O responsável pode já estar trabalhando nesse problema. Para prosseguir, revise o fluxo de trabalho atual e adicione a flag clippy ` D clippy::arithmetic side effects`. Consulte a documentação do clippy vinculada na issue para obter detalhes.
Pilha de tecnologia
rust
Domain
ci cddeveloper experience
Tipo Issue
Manutenção
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.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Bloqueado
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
RustCI configurationclippy
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50