FuelLabs/fuel-core

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

Aberta

#2.382 aberto em 22 de out. de 2024

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (1 responsável)Rust (2.861 forks)batch import
good first issue

Métricas do repositório

Stars
 (57.084 estrelas)
Métricas de merge de PR
 (Mesclagem média 8h 53m) (1 fundiu PR em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Revise a discussão referenciada no PR #2364, identifique todos os locais onde ExecutionGasPrice e DaGasPrice são usados como u64 e introduza wrappers de novos tipos para reforçar a segurança de tipos.
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.
Meio dia
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
RustGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65