-
Notifications
You must be signed in to change notification settings - Fork 87
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
f32/f64/usize in cosmwasm(and other no f32/f64 envs) reproduction and fixes #894
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #894 +/- ##
==========================================
- Coverage 67.67% 67.51% -0.16%
==========================================
Files 131 131
Lines 16079 16091 +12
==========================================
- Hits 10881 10864 -17
- Misses 5198 5227 +29
☔ View full report in Codecov by Sentry. |
Hey @tzemanovic, Could you please take a look at this branch and let me know if it fits your needs? Thanks! |
It does, thank you! |
Co-authored-by: Romain Ruetschi <romain@informal.systems> Signed-off-by: Farhad Shabani <Farhad.Shabani@gmail.com>
… fixes (#894) * fixed f64/f32 validation in wasm * what to do with serde json? main one is float, fork from cw is std, my fork pr not merged for no std * apply some adjustments + update cargo.lock for cw-check * deps: bump serde-json-wasm to v1.0.0 * chore: add unclog * ci: add cw-check * fix: remove cachix * fix: not needed to install/cache Rust in CI * imp: run CI only on releases and main branch * nit: on pull_request * nit: max_history_size as u64 * test: calculate_block_delay * nit: remove u64::max * Apply suggestions from code review Co-authored-by: Romain Ruetschi <romain@informal.systems> Signed-off-by: Farhad Shabani <Farhad.Shabani@gmail.com> --------- Signed-off-by: Farhad Shabani <Farhad.Shabani@gmail.com> Co-authored-by: dzmitry-lahoda <dzmitry@lahoda.pro> Co-authored-by: Romain Ruetschi <romain@informal.systems>
Closes: #850
Supersedes: #803
Remark
PR author checklist:
unclog
.docs/
).Reviewer checklist:
Files changed
in the GitHub PR explorer.