-
Notifications
You must be signed in to change notification settings - Fork 14
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
Polkadot update v0.9.42 #175
Polkadot update v0.9.42 #175
Conversation
Did the required followup changes, but now there is an internal substrate error: The error originates from Currently investigating why this happened. |
fc03e5b
to
4cb528d
Compare
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.
I could imagine that the error comes from divergent substrate versions.
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.
All of this looks good to me. All the fun polkadot update stuff.
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.
Nice, looks good to me! Now we just have to fix the test.
…existential deposit (ED) Co-authored-by: coax1d <andrewburger1991@gmail.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 to me!
See the following substrate PRs for details:
Weight::from_{ref_time, proof_size}
paritytech/substrate#13475For the last one I opted for suppressing the warning, but will open an issue for benchmarking it.
Closes #172