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

[JENKINS-72054] Add an option to skip the time-consuming post-processing step #1651

Merged
merged 6 commits into from
Jan 9, 2024

Conversation

uhafner
Copy link
Member

@uhafner uhafner commented Jan 8, 2024

Some projects noticed a long runtime of the post processing step after the issue parsing has been finished. It makes sense to make all of those steps optional (parameter), see also JENKINS-71492 for details.

@uhafner uhafner added the enhancement Enhancement of existing functionality label Jan 8, 2024
Copy link

codecov bot commented Jan 9, 2024

Codecov Report

Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (a698285) 81.83% compared to head (84d3d48) 81.67%.

Files Patch % Lines
...plugins/analysis/core/steps/ScanForIssuesStep.java 33.33% 2 Missing and 2 partials ⚠️
.../plugins/analysis/core/steps/RecordIssuesStep.java 50.00% 2 Missing ⚠️
...ns/plugins/analysis/core/steps/IssuesRecorder.java 80.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1651      +/-   ##
============================================
- Coverage     81.83%   81.67%   -0.16%     
- Complexity     1411     1414       +3     
============================================
  Files           249      249              
  Lines          5361     5376      +15     
  Branches        399      402       +3     
============================================
+ Hits           4387     4391       +4     
- Misses          861      870       +9     
- Partials        113      115       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@uhafner uhafner merged commit f5af613 into main Jan 9, 2024
33 of 35 checks passed
@uhafner uhafner deleted the skipPostProcessing branch January 9, 2024 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement of existing functionality
Projects
None yet
1 participant