Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Bump golemfactory/wasm to 0.6.0 #5089

Merged
merged 1 commit into from
Feb 7, 2020
Merged

Bump golemfactory/wasm to 0.6.0 #5089

merged 1 commit into from
Feb 7, 2020

Conversation

kubkon
Copy link
Contributor

@kubkon kubkon commented Feb 6, 2020

This PR bumps our Wasm sandbox to latest release 0.6.0 in Docker Hub.
The corresponding changes can be found at golemfactory/sp-wasm#38.
For completeness though, here's the summary of major changes to the sandbox:

  • port Reqc's change to VM which now makes BINARYEN_ASYNC_COMPILATION flag
    obsolete
  • refactors the CLI interface for the sandbox (changes clap for structopt)
  • refactors error handling (changes failure for thiserror)

This PR bumps our Wasm sandbox to latest release 0.6.0 in Docker Hub.
The corresponding changes can be found at golemfactory/sp-wasm#38.
For completeness though, here's the summary of major changes to the sandbox:
* port Reqc's change to VM which now makes `BINARYEN_ASYNC_COMPILATION` flag
  obsolete
* refactors the CLI interface for the sandbox (changes `clap` for `structopt`)
* refactors error handling (changes `failure` for `thiserror`)
@codecov
Copy link

codecov bot commented Feb 6, 2020

Codecov Report

Merging #5089 into b0.22 will increase coverage by <.01%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           b0.22   #5089      +/-   ##
========================================
+ Coverage     90%     90%   +<.01%     
========================================
  Files        234     234              
  Lines      22022   22022              
========================================
+ Hits       19821   19822       +1     
+ Misses      2201    2200       -1

Copy link
Contributor

@kmazurek kmazurek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rage

Copy link
Contributor

@jiivan jiivan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge after prints are removed

Copy link
Contributor

@jiivan jiivan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gang_olsena

@kubkon
Copy link
Contributor Author

kubkon commented Feb 7, 2020

gang_olsena

Am I the one getting hit with the brick by accident (or was it?)? :D

@kubkon
Copy link
Contributor Author

kubkon commented Feb 7, 2020

It's ALIIIIIVEEEE! Thanks @maaktweluit! 🎉

@jiivan jiivan merged commit 833d6e5 into b0.22 Feb 7, 2020
@jiivan jiivan deleted the kubkon/wasm-0.6.0 branch February 7, 2020 14:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants