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

vm-12 #4169

Merged
merged 3 commits into from
Jun 7, 2022
Merged

vm-12 #4169

merged 3 commits into from
Jun 7, 2022

Conversation

sasurobert
Copy link
Contributor

vm-12

andreibancioiu
andreibancioiu previously approved these changes Jun 6, 2022
alyn509
alyn509 previously approved these changes Jun 6, 2022
mariusmihaic
mariusmihaic previously approved these changes Jun 6, 2022
iulianpascalau
iulianpascalau previously approved these changes Jun 6, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jun 6, 2022

Codecov Report

Merging #4169 (f1db1e2) into master (68dadf3) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4169   +/-   ##
=======================================
  Coverage   74.83%   74.84%           
=======================================
  Files         609      609           
  Lines       80970    80970           
=======================================
+ Hits        60597    60600    +3     
+ Misses      15737    15735    -2     
+ Partials     4636     4635    -1     
Impacted Files Coverage Δ
storage/txcache/txListBySenderMap.go 97.50% <0.00%> (-2.50%) ⬇️
process/block/baseProcess.go 69.27% <0.00%> (+0.18%) ⬆️
p2p/libp2p/netMessenger.go 75.70% <0.00%> (+0.42%) ⬆️

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 68dadf3...f1db1e2. Read the comment docs.

iulianpascalau
iulianpascalau previously approved these changes Jun 6, 2022
dragos-rebegea
dragos-rebegea previously approved these changes Jun 6, 2022
SebastianMarian
SebastianMarian previously approved these changes Jun 6, 2022
ccorcoveanu
ccorcoveanu previously approved these changes Jun 6, 2022
andreibancioiu
andreibancioiu previously approved these changes Jun 6, 2022
Copy link
Contributor

@iulianpascalau iulianpascalau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

validated on import-db

Copy link
Collaborator

@gabi-vuls gabi-vuls left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

System test passed.

@iulianpascalau iulianpascalau merged commit 34afbe4 into master Jun 7, 2022
@iulianpascalau iulianpascalau deleted the vm-12 branch June 7, 2022 11:17
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