FuelLabs/fuel-core

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

Open

#2.382 aberto em 22 de out. de 2024

Ver no GitHub
 (2 comments) (0 reactions) (1 assignee)Rust (2.861 forks)batch import
good first issue

Métricas do repositório

Stars
 (57.084 stars)
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 na PR #2364, especificamente o comentário que sugeriu separar os tipos. Em seguida, identifique todas as ocorrências de `u64` representando preços de gás na base de código, distinguindo entre preço de gás de execução e preço de gás DA. Crie dois novos tipos, por exemplo, `ExecutionGasPrice` e `DaGasPrice`, implementando os traços necessários (ex.: aritmética, serialização), e refatore todos os usos para utilizá los. Consulte a PR vinculada para contexto adicional e possíveis armadilhas.
Pilha de tecnologia
rust
Domain
backendblockchain
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.
Bloqueado
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Basic Rust knowledgeUnderstanding of gas pricing in FuelFamiliarity with newtype pattern
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40