Skip to content

Conversation

Lredhdx
Copy link
Contributor

@Lredhdx Lredhdx commented Dec 15, 2021

What does this PR do?

Why are these changes required?

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@codecov-commenter
Copy link

Codecov Report

Merging #4187 (a5cc716) into release_v4.4.2 (4af63b5) will increase coverage by 0.00%.
The diff coverage is 60.00%.

Impacted file tree graph

@@                Coverage Diff                @@
##             release_v4.4.2    #4187   +/-   ##
=================================================
  Coverage             53.08%   53.09%           
- Complexity             7639     7641    +2     
=================================================
  Files                   812      812           
  Lines                 46167    46166    -1     
  Branches               5054     5054           
=================================================
+ Hits                  24510    24514    +4     
+ Misses                19728    19726    -2     
+ Partials               1929     1926    -3     
Impacted Files Coverage Δ
actuator/src/main/java/org/tron/core/vm/Op.java 75.00% <60.00%> (-1.48%) ⬇️
.../java/org/tron/common/overlay/server/SyncPool.java 39.31% <0.00%> (+0.85%) ⬆️
...ain/java/org/tron/consensus/dpos/StateManager.java 52.08% <0.00%> (+4.16%) ⬆️
...mon/overlay/discover/table/DistanceComparator.java 100.00% <0.00%> (+20.00%) ⬆️

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 4af63b5...a5cc716. Read the comment docs.

Copy link
Contributor

@yanghang8612 yanghang8612 left a comment

Choose a reason for hiding this comment

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

LGTM

@yanghang8612 yanghang8612 merged commit b6e343f into tronprotocol:release_v4.4.2 Dec 15, 2021
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.

3 participants