Tracker method calling part needs to be refactored
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 25/100
- Tipo di issue
- Refactoring
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- rust
- Ambito
- blockchain
Direzione di ricerca
Inizia confrontando le implementazioni di tracker() in types/src/transaction/shard.rs, types/src/transaction/action.rs e types/src/transaction/transaction.rs, quindi esamina le funzioni chiamanti correlate e le modifiche di PR #1548. Traccia le conversioni esistenti da Action a ShardTransaction e il percorso WrapCCC. Il lavoro è completo quando l’API tracker e i suoi chiamanti hanno una semantica coerente senza conversioni ridondanti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Rust
- Stelle
- 256
- Fork
- 50
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di CodeChain-io/codechain
-
is this project still alive? Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
CodeChain-io/codechain#2002 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
CodeChain-io/codechain#1993 · 2 commenti ·
-
Terminate gracefully Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
CodeChain-io/codechain#1987 · 1 commento ·
-
performance rpc
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
CodeChain-io/codechain#1973 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
CodeChain-io/codechain#1971 ·
Tutte le issue di CodeChain-io/codechain
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
state:needs triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
zed-industries/zed#64680 · 2 commenti ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
RustPython/RustPython#8802 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
TheLarkInn/aipm#2390 ·