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

Move code to new Coverage plugin #795

Merged
merged 7 commits into from
Oct 23, 2023
Merged

Move code to new Coverage plugin #795

merged 7 commits into from
Oct 23, 2023

Conversation

uhafner
Copy link
Member

@uhafner uhafner commented Oct 19, 2023

There is a lot of old and deprecated code in this plugin. Additionally, some steps have been deprecated. In order to simplify future development, it makes sense to split the code into two plugins. This is similar to the split of the old and deprecated warnings plugin into the new warnings next generation plugin.

The new code will be placed into the new Jenkins Coverage plugin.

@uhafner uhafner added the internal Internal changes without user or API impact label Oct 19, 2023
@codecov
Copy link

codecov bot commented Oct 23, 2023

Codecov Report

Merging #795 (0514f0d) into master (9d6fbe3) will decrease coverage by 1.73%.
The diff coverage is n/a.

❗ Current head 0514f0d differs from pull request most recent head fd1b6a2. Consider uploading reports for the commit fd1b6a2 to get more accurate results

@@             Coverage Diff              @@
##             master     #795      +/-   ##
============================================
- Coverage     74.58%   72.86%   -1.73%     
+ Complexity     1754     1032     -722     
============================================
  Files           135       88      -47     
  Lines          6394     3825    -2569     
  Branches        688      440     -248     
============================================
- Hits           4769     2787    -1982     
+ Misses         1400      892     -508     
+ Partials        225      146      -79     

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

@uhafner uhafner marked this pull request as ready for review October 23, 2023 20:18
@uhafner uhafner merged commit 8aedf0e into master Oct 23, 2023
3 of 12 checks passed
@uhafner uhafner deleted the coverage-plugin-split branch October 23, 2023 20:18
@uhafner uhafner restored the coverage-plugin-split branch October 23, 2023 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Internal changes without user or API impact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant