Skip to content

Conversation

@forfreeday
Copy link
Contributor

…version code:16580

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 #4201 (b16787f) into master (269ed47) will increase coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #4201      +/-   ##
============================================
+ Coverage     53.04%   53.10%   +0.05%     
- Complexity     7630     7641      +11     
============================================
  Files           812      812              
  Lines         46170    46170              
  Branches       5056     5056              
============================================
+ Hits          24492    24517      +25     
+ Misses        19747    19727      -20     
+ Partials       1931     1926       -5     
Impacted Files Coverage Δ
...mework/src/main/java/org/tron/program/Version.java 50.00% <ø> (ø)
...java/org/tron/core/capsule/TransactionCapsule.java 50.85% <0.00%> (+0.24%) ⬆️
.../java/org/tron/common/overlay/server/SyncPool.java 39.31% <0.00%> (+0.85%) ⬆️
.../main/java/org/tron/core/db2/common/TxCacheDB.java 69.13% <0.00%> (+1.23%) ⬆️
...mework/src/main/java/org/tron/core/db/Manager.java 52.82% <0.00%> (+1.23%) ⬆️
...src/main/java/org/tron/core/db/PendingManager.java 54.54% <0.00%> (+18.18%) ⬆️
...g/tron/core/exception/DupTransactionException.java 50.00% <0.00%> (+50.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 269ed47...b16787f. Read the comment docs.

@forfreeday forfreeday merged commit 744b0b8 into tronprotocol:master Dec 16, 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.

4 participants