bluealloy/revm

Simplify EthFrame implementation

Open

#1 886 ouverte le 5 déc. 2024

Voir sur GitHub
 (6 commentaires) (0 réactions) (0 assignés)Rust (1 019 forks)user submission
good first issue

Métriques du dépôt

Stars
 (2 194 stars)
Métriques de merge PR
 (Merge moyen 1j 14h) (32 PRs mergées en 30 j)

Description

Frame trait seems great, but implementation for Eth seem bloated. find the way how to make it nicer and more manageable

Guide contributeur

Direction de recherche
Le problème demande de simplifier l'implémentation d'EthFrame, actuellement perçue comme gonflée. Commencez par examiner le code actuel d'EthFrame dans la source de revm (probablement sous crates/revm/src/ ou similaire). Regardez la définition du trait Frame et comment EthFrame l'implémente. Passez en revue les 6 commentaires sur le problème pour comprendre les orientations proposées ou les préoccupations. Envisagez de créer un document de conception décrivant un plan pour découpler ou modulariser l'implémentation. Aucune PR n'est liée, donc la première contribution pourrait être un commentaire de recherche résumant les découvertes et proposant une approche de refactorisation.
Stack technique
rust
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60