Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Feat/add wasm support to dockerfile #984

Merged
merged 3 commits into from
Feb 25, 2022
Merged

Feat/add wasm support to dockerfile #984

merged 3 commits into from
Feb 25, 2022

Conversation

niccoloraspa
Copy link
Member

@niccoloraspa niccoloraspa commented Feb 25, 2022 •
edited
Loading

Description

This PR updates the Dockerfile to include the missing libraries needed for CosmWasm.

For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

@codecov-commenter
Copy link

Codecov Report

Merging #984 (71b0601) into main (e10e67e) will not change coverage.
The diff coverage is n/a.

❗ Current head 71b0601 differs from pull request most recent head 7fed84a. Consider uploading reports for the commit 7fed84a to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #984   +/-   ##
=======================================
  Coverage   20.60%   20.60%           
=======================================
  Files         193      193           
  Lines       25232    25232           
=======================================
  Hits         5200     5200           
  Misses      19080    19080           
  Partials      952      952

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e10e67e...7fed84a. Read the comment docs.

ValarDragon
ValarDragon approved these changes Feb 25, 2022
View reviewed changes
Copy link
Member

@ValarDragon ValarDragon left a comment

Choose a reason for hiding this comment

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

TY!

@ValarDragon ValarDragon merged commit 0202e20 into osmosis-labs:main Feb 25, 2022
@ValarDragon ValarDragon added the A:backport/v7.x Do not use. backport patches to v7.x branch label Feb 26, 2022
mergify bot pushed a commit that referenced this pull request Feb 26, 2022
Feat/add wasm support to dockerfile (#984)
f308d8b 
* Add wasm dependencies to Dockerfile

* Update CHANGELOG.md

(cherry picked from commit 0202e20)
@mergify mergify bot mentioned this pull request Feb 26, 2022
Merged
ValarDragon pushed a commit that referenced this pull request Feb 26, 2022
@mergify
Feat/add wasm support to dockerfile (#984) (#991)
2c990ef 
* Add wasm dependencies to Dockerfile

* Update CHANGELOG.md

(cherry picked from commit 0202e20)

Co-authored-by: Niccolo Raspa <nikever@users.noreply.github.com>
@ko0f ko0f mentioned this pull request Mar 3, 2022
Closed
@niccoloraspa niccoloraspa deleted the feat/add-wasm-support-to-dockerfile branch March 16, 2022 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ValarDragon ValarDragon ValarDragon approved these changes

Assignees
No one assigned
Labels
A:backport/v7.x Do not use. backport patches to v7.x branch
Projects
Osmosis Chain Development
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@niccoloraspa @codecov-commenter @ValarDragon