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

Draft+2911 #3194

Closed
wants to merge 221 commits into from
Closed

Draft+2911 #3194

wants to merge 221 commits into from

Conversation

brentstone
Copy link
Collaborator

Describe your changes

Indicate on which release or other PRs this topic is based on

Checklist before merging to draft

  • I have added a changelog
  • Git history is in acceptable state

grarco and others added 28 commits May 3, 2024 17:08
Co-authored-by: Gianmarco Fraccaroli <[email protected]>
* tiago/remove-build-warning:
  Fix unused import build warnings
* fraccaman+grarco/remove-benchmark:
  clippy,fmt
  removed outdated benchmarks
* fraccaman/rework-balance:
  Changelog for #3171
  Fix integration tests
  Standardize no balance response
  Improve error msg when no balance owner is found
  Remove `get_token_balance` from apps
  Remove IBC token query command
  Refactor the CLI's balance command
* grarco/remove-delta-map:
  Changelog #3172
  Removes `delta_map` from the `ShieldedContext` and all the code to work with the balance keys
  Removes `query_tx_deltas`
  Changelog #2675
  Removes unused errors and integration test
  Fixes econding. Updates shielded keys and addresses for tests
  Adjusts wasm txs and client args
  Removes masp pin key
  Rmoves old allowlisted gas
* McDaan-validator-alias:
  change to validator name (from validator alias)
  fix tests
  Changelog
  generating localnet files
  add validator alias field to metadata
Copy link

codecov bot commented May 7, 2024

Codecov Report

Attention: Patch coverage is 57.64895% with 1052 lines in your changes are missing coverage. Please review.

Project coverage is 60.12%. Comparing base (9d4de02) to head (4cf695a).
Report is 12 commits behind head on main.

Files Patch % Lines
crates/ibc/src/event.rs 46.94% 113 Missing ⚠️
...rates/apps/src/lib/node/ledger/shell/governance.rs 13.67% 101 Missing ⚠️
...tes/apps/src/lib/node/ledger/shell/testing/node.rs 0.00% 89 Missing ⚠️
crates/events/src/extend.rs 74.26% 88 Missing ⚠️
crates/sdk/src/tx.rs 0.00% 83 Missing ⚠️
crates/ibc/src/context/client.rs 17.10% 63 Missing ⚠️
crates/events/src/lib.rs 72.68% 62 Missing ⚠️
crates/sdk/src/masp.rs 0.00% 54 Missing ⚠️
crates/governance/src/event.rs 54.38% 52 Missing ⚠️
crates/proof_of_stake/src/storage.rs 5.55% 34 Missing ⚠️
... and 45 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3194      +/-   ##
==========================================
+ Coverage   59.40%   60.12%   +0.71%     
==========================================
  Files         298      303       +5     
  Lines       92326    92902     +576     
==========================================
+ Hits        54849    55857    +1008     
+ Misses      37477    37045     -432     

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

@brentstone brentstone closed this May 7, 2024
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.

9 participants