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-58383] [JENKINS-57377] Add configuration of aggregation chart #211

Merged
merged 11 commits into from
Sep 18, 2019

Conversation

uhafner
Copy link
Member

@uhafner uhafner commented Sep 13, 2019

Adds the options to show the aggregation on the top or bottom of all charts, or to hide the chart.

@uhafner uhafner added the enhancement Enhancement of existing functionality label Sep 13, 2019
@codecov
Copy link

codecov bot commented Sep 14, 2019

Codecov Report

Merging #211 into master will increase coverage by 0.06%.
The diff coverage is 89.79%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #211      +/-   ##
============================================
+ Coverage     82.68%   82.74%   +0.06%     
- Complexity     1552     1565      +13     
============================================
  Files           245      246       +1     
  Lines          5312     5338      +26     
  Branches        383      387       +4     
============================================
+ Hits           4392     4417      +25     
  Misses          775      775              
- Partials        145      146       +1
Impacted Files Coverage Δ Complexity Δ
...plugins/analysis/core/steps/PublishIssuesStep.java 52.41% <100%> (-0.65%) 23 <2> (+2)
...ns/plugins/analysis/core/util/ModelValidation.java 84.37% <100%> (+1.32%) 31 <1> (+1) ⬆️
...alysis/core/util/AggregationTrendChartDisplay.java 100% <100%> (ø) 1 <1> (?)
.../plugins/analysis/core/steps/RecordIssuesStep.java 54.49% <66.66%> (+0.42%) 37 <1> (+1) ⬆️
...ns/plugins/analysis/core/steps/IssuesRecorder.java 82% <75%> (-0.39%) 80 <3> (+2)
...s/plugins/analysis/core/steps/IssuesPublisher.java 97.53% <85.71%> (-1.14%) 28 <0> (+3)
...enkins/plugins/analysis/core/util/QualityGate.java 50% <0%> (-2.05%) 9% <0%> (-1%)
...ns/plugins/analysis/core/model/AnalysisResult.java 86.06% <0%> (-0.61%) 59% <0%> (-1%)
...ins/plugins/analysis/warnings/tasks/OpenTasks.java 40.29% <0%> (+5.97%) 10% <0%> (+4%) ⬆️
... and 1 more

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 f585409...3852c10. Read the comment docs.

@jenkinsci jenkinsci deleted a comment Sep 14, 2019
@jenkinsci jenkinsci deleted a comment Sep 14, 2019
@jenkinsci jenkinsci deleted a comment Sep 15, 2019
@jenkinsci jenkinsci deleted a comment Sep 15, 2019
@jenkinsci jenkinsci deleted a comment Sep 16, 2019
@jenkinsci jenkinsci deleted a comment Sep 17, 2019
@uhafner uhafner merged commit 7be203a into master Sep 18, 2019
@uhafner uhafner deleted the chart-configuration branch September 18, 2019 12:56
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
Development

Successfully merging this pull request may close these issues.

1 participant