Tracker method calling part needs to be refactored
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Refactorización
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- rust
- Área
- blockchain
Línea de trabajo
Comienza comparando las implementaciones de tracker() en types/src/transaction/shard.rs, types/src/transaction/action.rs y types/src/transaction/transaction.rs; después, inspecciona las funciones llamadoras relacionadas y los cambios de PR #1548. Sigue las conversiones existentes de Action-to-ShardTransaction y la ruta WrapCCC. Se considera terminado cuando la API de tracker y sus llamadores tienen una semántica coherente sin conversiones redundantes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Now the tracker() method can be called through the structs ShardTransaction, Action and Transaction.
Some caller function code is now using useless conversion from Action to ShardTransaction though the conversion was already implemented inside of the tracker() of Action.
As well as the above, after the pr#1548, the implementations will become inconsistent. Previously the tracker was needed only for asset transactions, but now the WrapCCC needs tracker though it's not an asset transaction.
From this semantic inconsistency, there is an implementation gap between tracker() of Action and the others. Therefore, the refactoring needs for tracker() and related caller functions not to be confused in the future.
- Lenguaje dominante
- Rust
- Estrellas
- 256
- Forks
- 50
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de CodeChain-io/codechain
-
is this project still alive? Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
CodeChain-io/codechain#2002 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
CodeChain-io/codechain#1993 · 2 comentarios ·
-
Terminate gracefully Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
CodeChain-io/codechain#1987 · 1 comentario ·
-
performance rpc
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
CodeChain-io/codechain#1973 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
CodeChain-io/codechain#1971 ·
Todos los issues de CodeChain-io/codechain
Issues similares
-
bug github_actions
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
registrystack/registry-stack#1393 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
longbridge/gpui-kit#3223 ·
-
bug engine
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
rocky-data/rocky#2181 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
oasisprotocol/oasis-sdk#2523 ·
-
[indexer] [QA] Add a focused test for the new NonRetryableError / assertSocketAlive() behavior. Abiertobot:ai-assisted component:indexer QA-roadmap status:untriaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
midnightntwrk/midnight-indexer#1557 ·