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.

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

V0.45.x osmo cw1point1 #1009

Closed
wants to merge 14 commits into from

Conversation

faddat
Copy link
Contributor

@faddat faddat commented Sep 19, 2022

This PR is one of the two possible implementations for

Specifically, this updates the osmosis-flavored wasmd to match the main branch of wasmd, which uses cosmwasm 1.1

@faddat faddat requested a review from alpe as a code owner September 19, 2022 11:14
go.mod Outdated Show resolved Hide resolved
@alpe alpe added the spike Demo to showcase an idea. label Sep 19, 2022
@codecov
Copy link

codecov bot commented Sep 19, 2022

Codecov Report

Merging #1009 (d88285e) into main (a76999a) will increase coverage by 0.61%.
The diff coverage is 25.00%.

❗ Current head d88285e differs from pull request most recent head dc659cd. Consider uploading reports for the commit dc659cd to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1009      +/-   ##
==========================================
+ Coverage   59.68%   60.29%   +0.61%     
==========================================
  Files          52       52              
  Lines        6586     6438     -148     
==========================================
- Hits         3931     3882      -49     
+ Misses       2367     2267     -100     
- Partials      288      289       +1     
Impacted Files Coverage Δ
app/test_helpers.go 0.78% <0.00%> (+<0.01%) ⬆️
x/wasm/client/cli/genesis_msg.go 68.97% <0.00%> (-2.46%) ⬇️
x/wasm/keeper/snapshotter.go 62.85% <0.00%> (-1.85%) ⬇️
x/wasm/keeper/keeper.go 87.30% <50.00%> (-0.46%) ⬇️
x/wasm/keeper/contract_keeper.go 92.85% <0.00%> (-2.15%) ⬇️
x/wasm/keeper/genesis.go 88.57% <0.00%> (-0.77%) ⬇️
x/wasm/module.go 48.93% <0.00%> (-0.52%) ⬇️
x/wasm/types/keys.go 51.11% <0.00%> (ø)
... and 10 more

@alpe
Copy link
Contributor

alpe commented Sep 28, 2022

Thanks for the spike. I will close this now in favour of #1022
See: #941 (comment)

@alpe alpe closed this Sep 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spike Demo to showcase an idea.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants