-
Notifications
You must be signed in to change notification settings - Fork 783
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
XCM v6 #7123
base: master
Are you sure you want to change the base?
XCM v6 #7123
Conversation
This is much more complicated than I initially anticipated. The |
most of the Xcm v5 code are written without any consideration of future compatibility, and now someone have to refactor all of it... |
approach looks good! @xlc if you don't have the bandwidth to do this to completion (likely), I'm sure we can find someone in the ecosystem to do it, especially if you provide a good plan/framework/starting-point - let me know |
@acatangiu that will be great. I will spend a few more days to do the core refactoring and then someone can help fix integrations with different runtimes and remaining tests |
closes #7095
Setup XCM v6 but with a more extensible approach. This should be 100% ABI with XCM v5 currently but need tests to assert that.
This is only a (not so) quick draft and looking for initial feedbacks.
Ideally, in future XCM versions, we only need to implement the new instructions without the need to duplicate all the existing ones.
TODOs:
Changes: