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

feat(config): update jacoco config #5475

Merged

Conversation

forfreeday
Copy link
Contributor

What does this PR do?
updating code coverage statistics configuration

Why are these changes required?
optimize code coverage statistics

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@halibobo1205
Copy link
Contributor

@forfreeday Please link an issue, as detailed as possible.

@forfreeday
Copy link
Contributor Author

@halibobo1205
OK

@forfreeday forfreeday linked an issue Sep 7, 2023 that may be closed by this pull request
@forfreeday forfreeday changed the base branch from develop to release_v4.3.0 September 12, 2023 09:27
@forfreeday forfreeday changed the base branch from release_v4.3.0 to release_v4.7.3 September 12, 2023 09:32
@codecov-commenter
Copy link

Codecov Report

Merging #5475 (2ce1fa7) into release_v4.7.3 (0fbbc7a) will increase coverage by 0.26%.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@                 Coverage Diff                  @@
##             release_v4.7.3    #5475      +/-   ##
====================================================
+ Coverage             64.62%   64.89%   +0.26%     
- Complexity             9686     9992     +306     
====================================================
  Files                   867      894      +27     
  Lines                 51725    53793    +2068     
  Branches               5755     5921     +166     
====================================================
+ Hits                  33426    34907    +1481     
- Misses                15695    16167     +472     
- Partials               2604     2719     +115     

see 35 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link

@aguinaldok4 aguinaldok4 left a comment

Choose a reason for hiding this comment

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

~~:+639773180017'Merging #5475 (2ce1fa7) into release_v4.7.3 (0fbbc7a) will increase coverage by 0.26%.
The diff coverage is_status.pg_stspg

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@ Coverage Diff @@ ## release_v4.7.3 #5475 +/- ## ==================================================== + Coverage 64.62% 64.89% +0.26% - Complexity 9686 9992 +306 ==================================================== Files 867 894 +27 Lines 51725 53793 +2068 Branches 5755 5921 +166 ==================================================== + Hits 33426 34907 +1481 - Misses 15695 16167 +472 - Partials 2604 2719 +115

~~

@halibobo1205
Copy link
Contributor

@forfreeday Please link an issue, as detailed as possible.

#5380, There is no detailed information on this change, please update it.

@forfreeday forfreeday linked an issue Sep 13, 2023 that may be closed by this pull request
@forfreeday forfreeday merged commit 9750847 into tronprotocol:release_v4.7.3 Sep 15, 2023
5 checks passed
@halibobo1205 halibobo1205 added this to the GreatVoyage-v4.7.3 milestone Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants