-
Notifications
You must be signed in to change notification settings - Fork 446
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
Stabilize seal_call
#1041
Stabilize seal_call
#1041
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1041 +/- ##
===========================================
+ Coverage 63.72% 78.74% +15.01%
===========================================
Files 248 248
Lines 9361 9361
===========================================
+ Hits 5965 7371 +1406
+ Misses 3396 1990 -1406
Continue to review full report at Codecov.
|
#[link(wasm_import_module = "__unstable__")] | ||
extern "C" { |
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.
This should've been removed in #1036, no? Do you need to merge master
?
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.
There is still seal_ecdsa_recover
which is unstable, but I just merged master
and there were some conflicts to resolve because of that PR, so good to merge nontheless.
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.
Thanks for the clarification 👍
🦑 📈 ink! Example Contracts ‒ Size Change Report 📉 🦑These are the results of building the
Link to the run | Last update: Wed Nov 24 21:52:56 CET 2021 |
Follow-up to paritytech/substrate#10366.