FuelLabs/fuel-core

Unify defaults for config

Open

#975 aperta il 2 feb 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Rust (2861 fork)batch import
fuel-coregood first issuetech-debt

Metriche repository

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

Descrizione

The cli run: https://github.com/FuelLabs/fuel-core/blob/3a88bececec821bf8ebbc33d001b84f34968a191/bin/fuel-core/src/cli/run.rs#L62 has a set of defaults that duplicate and are sometimes different to the actual defaults for the real config. We should unify these values so there is a single source of truth.

Guida contributor

Direzione di ricerca
Inizia esaminando i valori predefiniti definiti in bin/fuel core/src/cli/run.rs alla riga 62 e confrontali con i valori predefiniti effettivi per la struct config nella codebase. Identifica eventuali discrepanze o duplicazioni. Quindi proponi una refactoring per centralizzare i valori predefiniti in un'unica fonte di verità, possibilmente spostandoli in una posizione comune o utilizzando un'implementazione predefinita.
Tech stack
rust
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65