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

merge queue: embarking main (04b81b1) and #4133 together #4256

Closed
wants to merge 19 commits into from
Closed

merge queue: embarking main (04b81b1) and #4133 together #4256

wants to merge 19 commits into from

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Jan 20, 2025 •
edited
Loading

🎉 This pull request has been checked successfully and will be merged soon. 🎉

Branch main (04b81b1) and #4133 are embarked together for merge.

This pull request has been created by Mergify to speculatively check the mergeability of #4133.
You don't need to do anything. Mergify will close this pull request automatically when it is complete.

Required conditions of queue main-queue for merge:

  • #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • any of [🛡 GitHub branch protection]:
    • check-success = test-e2e-with-device-automation
    • check-neutral = test-e2e-with-device-automation
    • check-skipped = test-e2e-with-device-automation
  • any of [🛡 GitHub branch protection]:
    • check-success = check-benchmarks
    • check-neutral = check-benchmarks
    • check-skipped = check-benchmarks
  • any of [🛡 GitHub branch protection]:
    • check-success = check-packages
    • check-neutral = check-packages
    • check-skipped = check-packages
  • any of [🛡 GitHub branch protection]:
    • check-success = build-binaries
    • check-neutral = build-binaries
    • check-skipped = build-binaries
  • any of [🛡 GitHub branch protection]:
    • check-success = build-wasm (wasm-for-tests, build-wasm-tests-scripts, wasm_for_tests/*.wasm)
    • check-neutral = build-wasm (wasm-for-tests, build-wasm-tests-scripts, wasm_for_tests/*.wasm)
    • check-skipped = build-wasm (wasm-for-tests, build-wasm-tests-scripts, wasm_for_tests/*.wasm)
  • any of [🛡 GitHub branch protection]:
    • check-success = lints
    • check-neutral = lints
    • check-skipped = lints
  • any of [🛡 GitHub branch protection]:
    • check-success = rust-docs
    • check-neutral = rust-docs
    • check-skipped = rust-docs
  • any of [🛡 GitHub branch protection]:
    • check-success = test-wasm
    • check-neutral = test-wasm
    • check-skipped = test-wasm
  • any of [🛡 GitHub branch protection]:
    • check-success = test-e2e (0)
    • check-neutral = test-e2e (0)
    • check-skipped = test-e2e (0)
  • any of [🛡 GitHub branch protection]:
    • check-success = test-e2e (1)
    • check-neutral = test-e2e (1)
    • check-skipped = test-e2e (1)
  • any of [🛡 GitHub branch protection]:
    • check-success = test-e2e (2)
    • check-neutral = test-e2e (2)
    • check-skipped = test-e2e (2)
  • any of [🛡 GitHub branch protection]:
    • check-success = test-e2e (3)
    • check-neutral = test-e2e (3)
    • check-skipped = test-e2e (3)
  • any of [🛡 GitHub branch protection]:
    • check-success = test-e2e (4)
    • check-neutral = test-e2e (4)
    • check-skipped = test-e2e (4)
  • any of [🛡 GitHub branch protection]:
    • check-success = test-integration
    • check-neutral = test-integration
    • check-skipped = test-integration
  • any of [🛡 GitHub branch protection]:
    • check-success = test-unit
    • check-neutral = test-unit
    • check-skipped = test-unit
  • any of [🛡 GitHub branch protection]:
    • check-neutral = Mergify Merge Protections
    • check-skipped = Mergify Merge Protections
    • check-success = Mergify Merge Protections

Required conditions to stay in the queue:

---
previous_failed_batches: []
pull_requests:
  - number: 4133
...

sug0 and others added 19 commits January 20, 2025 15:34
@sug0 @batconjurer @tzemanovic
Update IBC middleware stack
9a35a3f 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @batconjurer @tzemanovic
Refactor PFM module impl
8056a78 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @batconjurer @tzemanovic
Fix IBC verifiers inclusion
e9fa612 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @batconjurer @tzemanovic
Convert from address to signer
f92bf28 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @batconjurer @tzemanovic
Redenominate amounts
da847b8 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @batconjurer @tzemanovic
Add new Osmosis related IBC memos
83893a7 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @batconjurer @tzemanovic
Implement shielded recv IBC middleware
278de0f 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @batconjurer @tzemanovic
Implement Osmosis swap SDK functionality
f5bbcdd 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @batconjurer @tzemanovic
Avoid full gen IBC shielding args clone
5aedb84 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @batconjurer @tzemanovic
Expose Osmosis swaps via CLI
0e87e70 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @batconjurer @tzemanovic
Accept shielded recv IBC memos in e2e tests
7933d96 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @batconjurer @tzemanovic
Add shielded recv e2e tests
dda46cf 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @batconjurer @tzemanovic
Add Osmosis contract bytecode fixtures
37f9af6 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @batconjurer @tzemanovic
Add Osmosis XCS e2e test
d8b07ce 
Co-authored-by: Jacob Turner <jacob.turner870@gmail.com>
@sug0 @tzemanovic
Expose SDK function to gen disposable signing keys
616f77c
@sug0 @tzemanovic
Generate disposable overflow receivers
573a9ab
@sug0 @tzemanovic
Improve Osmosis swaps CLI help msgs
e71c02b
@sug0 @tzemanovic
Changelog for #4133
1d5de2c
@mergify
Merge of #4133
18eda58
@codecov Codecov
Copy link

codecov bot commented Jan 20, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 9.69163% with 615 lines in your changes missing coverage. Please review.

Project coverage is 74.09%. Comparing base (04b81b1) to head (18eda58).
Report is 19 commits behind head on main.

Files with missing lines Patch % Lines
crates/sdk/src/rpc.rs 0.00% 268 Missing ⚠️
crates/sdk/src/args.rs 0.00% 193 Missing ⚠️
...rates/ibc/src/context/middlewares/shielded_recv.rs 22.85% 81 Missing ⚠️
crates/ibc/src/msg.rs 21.56% 40 Missing ⚠️
crates/sdk/src/tx.rs 0.00% 21 Missing ⚠️
crates/core/src/address.rs 0.00% 6 Missing ⚠️
crates/core/src/token.rs 0.00% 6 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #4256      +/-   ##
==========================================
- Coverage   74.54%   74.09%   -0.45%     
==========================================
  Files         344      345       +1     
  Lines      109256   109721     +465     
==========================================
- Hits        81448    81302     -146     
- Misses      27808    28419     +611

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

@mergify mergify bot closed this Jan 20, 2025
@mergify mergify bot deleted the mergify/merge-queue/2f9fe406c9 branch January 20, 2025 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@sug0