FuelLabs/fuel-core

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

Open

#2382 aperta il 22 ott 2024

Vedi su GitHub
 (2 commenti) (0 reazioni) (1 assegnatario)Rust (2861 fork)batch import
good first issue

Metriche repository

Star
 (57.084 star)
Metriche merge PR
 (Merge medio 8h 53m) (1 PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina la discussione sulla PR #2364, identifica tutti i punti in cui ExecutionGasPrice e DaGasPrice sono usati come u64 e introduci wrapper di nuovi tipi per rafforzare la sicurezza dei tipi.
Tech stack
rust
Dominio
blockchain
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65