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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate runBlock to typescript #512
Conversation
This pull request introduces 1 alert when merging 5e0f45e into 41e867e - view on LGTM.com new alerts:
Comment posted by LGTM.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
馃槃
Looks good, thanks!
@s1na If we target a VM |
@s1na @alcuadrado Thought a bit about VM releases yesterday. My idea was that we should now switch a bit the focus and concentrate on Along we continue with refactoring, but now on a more gradual pace, @s1na has actually shifted around A LOT of code. 馃構 I would also assume that there might be input from the community on the new structure once we have this out within an official For a How does this sound? |
We should be able to do the beta release by then. As you also pointed out in #479 there are some unclear things with the design, but we can solve them gradually. Documentation around the modified parts is also something I'll need to do.
Yeah, too much. This was a mistake. From now I'll change gears and take a more gradual approach. The plan sounds good. About ewasm, we'll probably know more in future as the scope of ewasm within ethereum 1.x is not fully clear at the moment. |
@s1na Ok, that sounds like a solid plan. Then let's do a |
Another one bites the dust 馃槃