FuelLabs/fuels-rs

Add clippy check for overflowing arithmetic

Open

#1.238 geöffnet am 14. Dez. 2023

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Rust (1.360 Forks)batch import
Securitygood first issuetech-debtv1.0

Repository-Metriken

Stars
 (43.211 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Die CI Konfiguration befindet sich wahrscheinlich in einer GitHub Actions Workflow Datei (z. B. .github/workflows/ci.yml). Der Beauftragte arbeitet möglicherweise bereits an diesem Issue. Um fortzufahren, überprüfe den aktuellen Workflow und füge das Clippy Flag ` D clippy::arithmetic side effects` hinzu. Einzelheiten findest du in der im Issue verlinkten Clippy Dokumentation.
Tech Stack
rust
Domain
ci cddeveloper experience
Issue Type
Wartung
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.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Blockiert
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
RustCI configurationclippy
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50