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

[mpm] Add version derive for mpm #3357

Merged
merged 1 commit into from
Apr 21, 2022
Merged

[mpm] Add version derive for mpm #3357

merged 1 commit into from
Apr 21, 2022

Conversation

jolestar
Copy link
Member

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: fix #3353

What is the new behavior?

Other information

@codecov
Copy link

codecov bot commented Apr 20, 2022

Codecov Report

Merging #3357 (317468c) into master (8a741cb) will decrease coverage by 0.25%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3357      +/-   ##
==========================================
- Coverage   32.11%   31.86%   -0.24%     
==========================================
  Files         500      500              
  Lines       46814    46814              
  Branches    21311    21311              
==========================================
- Hits        15028    14912     -116     
+ Misses      17737    17692      -45     
- Partials    14049    14210     +161     
Flag Coverage Δ
unittests 31.86% <ø> (-0.24%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
vm/types/src/on_chain_config/mod.rs 43.84% <0.00%> (-12.32%) ⬇️
chain/open-block/src/lib.rs 31.79% <0.00%> (-7.28%) ⬇️
executor/src/block_executor.rs 22.23% <0.00%> (-6.66%) ⬇️
state/state-store-api/src/lib.rs 43.75% <0.00%> (-6.25%) ⬇️
commons/forkable-jellyfish-merkle/src/lib.rs 41.77% <0.00%> (-4.98%) ⬇️
commons/accumulator/src/tree.rs 53.21% <0.00%> (-4.92%) ⬇️
consensus/src/consensus.rs 58.74% <0.00%> (-4.76%) ⬇️
chain/src/verifier/mod.rs 36.72% <0.00%> (-4.68%) ⬇️
state/state-tree/src/mock/mod.rs 43.48% <0.00%> (-4.34%) ⬇️
consensus/src/dummy.rs 69.57% <0.00%> (-4.34%) ⬇️
... and 57 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a741cb...317468c. Read the comment docs.

@jolestar jolestar merged commit 7b02ba5 into master Apr 21, 2022
@jolestar jolestar deleted the mpm_version branch April 21, 2022 01:49
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.

[Bug Report]mpm version
2 participants