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

Upgrade to sdk master 8b77cb #237

Closed
wants to merge 4 commits into from

Conversation

alpe
Copy link
Contributor

@alpe alpe commented Jul 29, 2020

Upgrade to lastest sdk

Note: there is a new LegacyQuerierHandler in modules that I could not set up proper due a lack of time. Some tests depend on this as query handler. I had to skip them.

@alpe alpe requested a review from ethanfrey as a code owner July 29, 2020 13:03
@codecov
Copy link

codecov bot commented Jul 29, 2020

Codecov Report

Merging #237 into 0.9.1_to_cosmos-sdk-0.39-master will decrease coverage by 12.65%.
The diff coverage is 14.06%.

Impacted file tree graph

@@                         Coverage Diff                          @@
##           0.9.1_to_cosmos-sdk-0.39-master     #237       +/-   ##
====================================================================
- Coverage                            32.42%   19.76%   -12.66%     
====================================================================
  Files                                   24       25        +1     
  Lines                                 3830     6419     +2589     
====================================================================
+ Hits                                  1242     1269       +27     
- Misses                                2521     5080     +2559     
- Partials                                67       70        +3     
Impacted Files Coverage Δ
x/wasm/internal/keeper/test_fuzz.go 90.90% <0.00%> (-9.10%) ⬇️
x/wasm/internal/types/query.pb.go 0.72% <ø> (ø)
x/wasm/internal/types/types.go 31.81% <ø> (ø)
x/wasm/internal/types/types.pb.go 0.55% <5.95%> (+<0.01%) ⬆️
x/wasm/internal/keeper/querier.go 24.00% <13.17%> (-12.14%) ⬇️
x/wasm/module.go 17.02% <18.18%> (+0.35%) ⬆️
app/app.go 91.90% <100.00%> (+0.07%) ⬆️
app/encoding.go 100.00% <100.00%> (ø)
x/wasm/internal/keeper/test_common.go 87.28% <100.00%> (+0.21%) ⬆️

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 cc37320...364aa78. Read the comment docs.

@alpe
Copy link
Contributor Author

alpe commented Aug 6, 2020

Obsolete by #247

@alpe alpe closed this Aug 6, 2020
@alpe alpe deleted the 0.9.1_to_master_8b77cb branch August 6, 2020 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant