FuelLabs/sway

Rename storage access methods

Open

#4740 aperta il 4 lug 2023

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)Rust (5421 fork)batch import
breakingcompiler: uienhancementgood first issuelib: stdstorageteam:compiler

Metriche repository

Star
 (61.702 star)
Metriche merge PR
 (Merge medio 22h 6m) (16 PR mergiate in 30 g)

Descrizione

StorageKey's read and try_read are confusing, notably because of errors related to uninitialized storage.

try_read should be the default behavior and there should be extensive documentation of what happens with the storage initialization.

try_read should be renamed to read and read to read_unchecked, but this is a breaking and abrasive change, so we should consider a way of migrating the interface that doesn't startle the users.

Guida contributor

Direzione di ricerca
Esamina i metodi read e try read correnti di StorageKey, comprendi la gestione degli errori per lo storage non inizializzato e implementa la ridenominazione con avvisi di deprecazione e un percorso di migrazione.
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.
3-5 giorni
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
RustSway
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60