FuelLabs/fuel-core

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

Open

#2.382 geöffnet am 22. Okt. 2024

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

Repository-Metriken

Stars
 (57.084 Stars)
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üfe die Diskussion in PR #2364, insbesondere den Kommentar, der die Trennung der Typen vorgeschlagen hat. Identifiziere dann alle Vorkommen von `u64`, die Gaspreise in der Codebasis darstellen, und unterscheide dabei zwischen Ausführungs Gaspreis und DA Gaspreis. Erstelle zwei neue Typen, z.B. `ExecutionGasPrice` und `DaGasPrice`, die notwendige Traits implementieren (z.B. Arithmetik, Serialisierung), und refaktoriere alle Verwendungen, um sie zu nutzen. Konsultiere den verlinkten PR für zusätzlichen Kontext und potenzielle Fallstricke.
Tech Stack
rust
Domain
backendblockchain
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.
Blockiert
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Basic Rust knowledgeUnderstanding of gas pricing in FuelFamiliarity with newtype pattern
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40