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

feat: bumpup ibc-go to v4 #61

Merged
merged 8 commits into from
Aug 3, 2023
Merged

feat: bumpup ibc-go to v4 #61

merged 8 commits into from
Aug 3, 2023

Conversation

loloicci
Copy link

@loloicci loloicci commented Aug 2, 2023

This PR merges CosmWasm#1088 and CosmWasm#1180 without ICA part.

How has this been tested?

unit tests (go test ./...)

Screenshots (if appropriate):

Checklist:

  • I followed the contributing guidelines and code of conduct.
  • I have added a relevant changelog to CHANGELOG.md
  • I have added tests to cover my changes. (not needed)
  • I have updated the documentation accordingly. (not needed)
  • I have updated API documentation client/docs/swagger-ui/swagger.yaml (not needed)

…#1088)

cherry-pick original CosmWasm#1088 except ICA part.

* Upgrade to IBC v4.2.0 with interchain-accounts v0.2.4

* Fix linter issues

* Test version handshake

* Add more ibc e2e scenarios

* Add fee middleware to wasm

* Get ibc app version from middleware
@loloicci loloicci requested a review from zemyblue August 2, 2023 10:53
@loloicci loloicci self-assigned this Aug 2, 2023
@loloicci loloicci changed the title feat: bumpup cosmwasm-ibc to v4 feat: bumpup ibc-go to v4 Aug 2, 2023
@codecov
Copy link

codecov bot commented Aug 2, 2023

Codecov Report

Merging #61 (3494e8b) into main (164efa4) will increase coverage by 1.16%.
The diff coverage is 74.02%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #61      +/-   ##
==========================================
+ Coverage   60.86%   62.02%   +1.16%     
==========================================
  Files          80       80              
  Lines        9894     9971      +77     
==========================================
+ Hits         6022     6185     +163     
+ Misses       3555     3450     -105     
- Partials      317      336      +19     
Files Changed Coverage Δ
app/ante.go 56.09% <ø> (ø)
app/test_access.go 0.00% <ø> (ø)
app/test_helpers.go 0.64% <0.00%> (-0.01%) ⬇️
x/wasm/keeper/handler_plugin.go 82.50% <ø> (ø)
x/wasm/keeper/handler_plugin_encoders.go 77.65% <ø> (ø)
x/wasm/keeper/ibc.go 75.00% <ø> (ø)
x/wasm/keeper/query_plugins.go 75.18% <ø> (ø)
x/wasm/ibc.go 50.86% <40.00%> (+42.06%) ⬆️
x/wasm/keeper/relay.go 96.42% <50.00%> (-0.94%) ⬇️
app/app.go 88.77% <100.00%> (+0.76%) ⬆️
... and 1 more

... and 2 files with indirect coverage changes

appplus/app.go Fixed Show fixed Hide fixed
go.mod Outdated Show resolved Hide resolved
go.mod Show resolved Hide resolved
@zemyblue zemyblue merged commit dedcd9e into main Aug 3, 2023
12 checks passed
@loloicci loloicci deleted the ibc4 branch August 3, 2023 05:28
@zemyblue zemyblue mentioned this pull request Oct 19, 2023
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.

6 participants