FuelLabs/fuel-core

Separate Execution Gas Price and DA Gas Price into separate newtypes to add clarity

Offen

#2.382 geöffnet am 22.10.2024

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Rust (2.861 Forks)batch import
good first issue

Repository-Metriken

Stars
 (57.084 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 8h 53m) (1 gemergte PR in 30 T)

Beschreibung

It's been suggested that we separate the two just to help organize the code a bit better. Instead of just dealing withu64 everywhere, it could help the reader to have types to communicate everywhere in the code the purpose of the numbers.

Contributor Guide

Research-Richtung
Überprüfen Sie die referenzierte Diskussion in PR #2364, identifizieren Sie alle Stellen, an denen ExecutionGasPrice und DaGasPrice als u64 verwendet werden, und führen Sie Newtype Wrapper ein, um die Typsicherheit zu erzwingen.
Tech Stack
rust
Domain
blockchain
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
RustGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65