-
Notifications
You must be signed in to change notification settings - Fork 91
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
Fix/release v0.11.0/v2 #1882
Fix/release v0.11.0/v2 #1882
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1882 +/- ##
==========================================
+ Coverage 46.74% 46.78% +0.03%
==========================================
Files 169 169
Lines 13176 13165 -11
==========================================
Hits 6159 6159
+ Misses 7017 7006 -11 ☔ View full report in Codecov by Sentry. |
Please wait with merging till we have sorted out the runtime version increment we wanna do here. |
pub const fn size_of_feed<T: Config>() -> u32 { | ||
sp_std::mem::size_of::<(T::OracleKey, T::OracleValue, MomentOf<T>)>() as u32 | ||
pub fn size_of_feed<T: Config>() -> u32 { | ||
let max_len = <(T::OracleKey, T::OracleValue, MomentOf<T>)>::max_encoded_len(); |
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.
👍🏻
Closing in favor of #1886 |
Description
Replaces #1881.
Changes and Descriptions
WasmExecutor
everywhereMaxEncodedLen
instead ofmem::size_of
Checklist: