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

Changes to EmbeddedVM #116

Closed
Tracked by #118
dr-orlovsky opened this issue Nov 21, 2022 · 0 comments · Fixed by #136
Closed
Tracked by #118

Changes to EmbeddedVM #116

dr-orlovsky opened this issue Nov 21, 2022 · 0 comments · Fixed by #136
Assignees
Labels
*consensus* Issues affecting distributed concensus epic Epic task covering multiple steps of implementation
Milestone

Comments

@dr-orlovsky
Copy link
Member

Change EmbededVm: it must provide a table linking metadata, state and state transitions to a specific embedded validation procedures. This will allow to avoid fixed set of field/state/transition type names defined at the consensus level and will much simplify the live of schema creators.

To have this changes in a non-breaking way a new Vm enum member will be added, with previous EmebddedVm renamed to LegacyVm. The LegacyVm will be deprecated and it will continue to rely on the pre-defined field types, which will not be exported anymore.

@dr-orlovsky dr-orlovsky added epic Epic task covering multiple steps of implementation *consensus* Issues affecting distributed concensus labels Nov 21, 2022
@dr-orlovsky dr-orlovsky self-assigned this Jan 23, 2023
@dr-orlovsky dr-orlovsky added this to the v0.10.0 milestone Jan 23, 2023
@dr-orlovsky dr-orlovsky moved this from Todo to Done in Releases & roadmap Feb 5, 2023
@dr-orlovsky dr-orlovsky linked a pull request Feb 6, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
*consensus* Issues affecting distributed concensus epic Epic task covering multiple steps of implementation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant