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 ibc-rs to v0.57 (backport #4247) #4310

Merged
merged 9 commits into from
Feb 4, 2025

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Feb 4, 2025

Describe your changes

NOTE: Need to update ibc-middleware with the release version

Mainly updated error types of ibc-rs. No logic should be changed.

Checklist before merging

  • If this PR has some consensus breaking changes, I added the corresponding breaking:: labels
    • This will require 2 reviewers to approve the changes
  • If this PR requires changes to the docs or specs, a corresponding PR is opened in the namada-docs repo
    • Relevant PR if applies:
  • If this PR affects services such as namada-indexer or namada-masp-indexer, a corresponding PR is opened in that repo
    • Relevant PR if applies:

This is an automatic backport of pull request #4247 done by [Mergify](https://mergify.com).

yito88 and others added 9 commits February 4, 2025 12:13
(cherry picked from commit 6240ef3)
(cherry picked from commit 4163024)
(cherry picked from commit d12d63d)
(cherry picked from commit 523a891)
(cherry picked from commit 3d916f2)
(cherry picked from commit c309a3e)
Co-authored-by: Tiago Carvalho <sugoiuguu@tfwno.gf>
(cherry picked from commit 12a0b0d)
(cherry picked from commit 759fcfc)
@tzemanovic tzemanovic added the merge Ready to merge - mergifyio bot will add the PR to merge queue when all checks pass label Feb 4, 2025
@mergify mergify bot merged commit 51cb210 into maint-libs-0.47 Feb 4, 2025
15 of 16 checks passed
@mergify mergify bot deleted the mergify/bp/maint-libs-0.47/pr-4247 branch February 4, 2025 12:19
Copy link

codecov bot commented Feb 4, 2025

Codecov Report

Attention: Patch coverage is 53.25843% with 208 lines in your changes missing coverage. Please review.

Project coverage is 74.16%. Comparing base (2561b0c) to head (58223a8).
Report is 11 commits behind head on maint-libs-0.47.

Files with missing lines Patch % Lines
crates/ibc/src/context/common.rs 56.56% 43 Missing ⚠️
crates/ibc/src/context/nft_transfer.rs 55.78% 42 Missing ⚠️
crates/ibc/src/lib.rs 9.67% 28 Missing ⚠️
crates/ibc/src/context/client.rs 0.00% 18 Missing ⚠️
crates/ibc/src/context/token_transfer.rs 75.00% 17 Missing ⚠️
crates/ibc/src/context/execution.rs 63.15% 14 Missing ⚠️
crates/ibc/src/context/nft_transfer_mod.rs 13.33% 13 Missing ⚠️
crates/ibc/src/context/validation.rs 72.50% 11 Missing ⚠️
crates/ibc/src/context/middlewares/pfm_mod.rs 0.00% 5 Missing ⚠️
crates/ibc/src/event.rs 0.00% 5 Missing ⚠️
... and 6 more
Additional details and impacted files
@@                 Coverage Diff                 @@
##           maint-libs-0.47    #4310      +/-   ##
===================================================
- Coverage            74.17%   74.16%   -0.01%     
===================================================
  Files                  345      345              
  Lines               110565   110603      +38     
===================================================
+ Hits                 82013    82032      +19     
- Misses               28552    28571      +19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge Ready to merge - mergifyio bot will add the PR to merge queue when all checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants