FuelLabs/sway

Rename storage access methods

Aberta

#4.740 aberto em 4 de jul. de 2023

Ver no GitHub
 (3 comentários) (0 reação) (0 responsável)Rust (5.421 forks)batch import
breakingcompiler: uienhancementgood first issuelib: stdstorageteam:compiler

Métricas do repositório

Stars
 (61.702 estrelas)
Métricas de merge de PR
 (Mesclagem média 22h 6m) (16 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Examine os métodos atuais read e try read do StorageKey, entenda o tratamento de erros para armazenamento não inicializado e implemente a renomeação com avisos de depreciação e um caminho de migração.
Pilha de tecnologia
rust
Domain
blockchain
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
3-5 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
RustSway
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60