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

fix: sweep broke lp returned events #4176

Merged
merged 9 commits into from Oct 31, 2023
Merged

Conversation

AlastairHolmes
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Oct 31, 2023

Codecov Report

Merging #4176 (df3e865) into main (f812372) will increase coverage by 0%.
Report is 7 commits behind head on main.
The diff coverage is 74%.

@@          Coverage Diff          @@
##            main   #4176   +/-   ##
=====================================
  Coverage     72%     72%           
=====================================
  Files        380     379    -1     
  Lines      61591   61667   +76     
  Branches   61591   61667   +76     
=====================================
+ Hits       44174   44241   +67     
- Misses     15119   15135   +16     
+ Partials    2298    2291    -7     
Files Coverage Δ
state-chain/pallets/cf-pools/src/benchmarking.rs 100% <100%> (ø)
state-chain/pallets/cf-pools/src/tests.rs 99% <100%> (ø)
state-chain/traits/src/mocks/lp_balance.rs 32% <ø> (ø)
api/lib/src/lp.rs 0% <0%> (ø)
state-chain/pallets/cf-pools/src/lib.rs 61% <82%> (+2%) ⬆️

... and 18 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@AlastairHolmes AlastairHolmes enabled auto-merge (squash) October 31, 2023 14:06
@AlastairHolmes AlastairHolmes merged commit e8584d1 into main Oct 31, 2023
41 checks passed
@AlastairHolmes AlastairHolmes deleted the fix/lp-api-returned-values branch October 31, 2023 14:14
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.

None yet

2 participants