Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Version and type sync with alloy #95

Closed
rkdud007 opened this issue Jun 21, 2024 · 0 comments · Fixed by #96
Closed

Version and type sync with alloy #95

rkdud007 opened this issue Jun 21, 2024 · 0 comments · Fixed by #96
Labels
optimization Improvements of performance or optimization P-middle Middle priority, for near future
Milestone

Comments

@rkdud007
Copy link
Member

alloy now have crate of v0.1.0 which means I need to consider removing the legacy dependecies, also can consider switching up on RpcType and ConsensusType.

But this can be able to happen when 2 case satify, otherwise will add redundant complexity :

  • type casting around RpcType -> ConsensusType should be working
  • rlp encode, decode of ConsensusType should be working

I will work on this if all 5 types I needed (header, account, storage, tx, receipt ) all satisfies the constraint above.

@rkdud007 rkdud007 added P-middle Middle priority, for near future optimization Improvements of performance or optimization labels Jun 21, 2024
@rkdud007 rkdud007 added this to the hdp v0.3.0 milestone Jun 21, 2024
@rkdud007 rkdud007 linked a pull request Jun 25, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
optimization Improvements of performance or optimization P-middle Middle priority, for near future
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant