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

Use new status icons in quality gate of pull request portlet #986

Merged
merged 1 commit into from
Jul 11, 2021

Conversation

uhafner
Copy link
Member

@uhafner uhafner commented Jul 10, 2021

Internally the quality gate still uses the BallColor class to visualize the quality gate status. This class has not been adapted to use the new Jenkins status icons so it is required to use a different way to show the correct icon of the quality gate.

  • Fix status icon in pull request portlet
  • Fix status icon in build summary (will be solved in another PR)

Screenshot

Bildschirmfoto 2021-07-10 um 14 06 57

@uhafner uhafner added the enhancement Enhancement of existing functionality label Jul 10, 2021
@codecov
Copy link

codecov bot commented Jul 10, 2021

Codecov Report

Merging #986 (383e208) into master (0001ba5) will decrease coverage by 0.06%.
The diff coverage is 56.55%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #986      +/-   ##
============================================
- Coverage     79.93%   79.87%   -0.07%     
- Complexity     1413     1447      +34     
============================================
  Files           237      238       +1     
  Lines          5199     5306     +107     
  Branches        406      416      +10     
============================================
+ Hits           4156     4238      +82     
- Misses          893      911      +18     
- Partials        150      157       +7     
Impacted Files Coverage Δ
...ugins/analysis/core/columns/IssuesTotalColumn.java 76.04% <0.00%> (ø)
...ins/analysis/core/model/AggregatedTrendAction.java 28.26% <0.00%> (-0.63%) ⬇️
...kins/plugins/analysis/core/model/FilesScanner.java 91.11% <ø> (ø)
...kins/plugins/analysis/core/model/IssuesDetail.java 56.98% <0.00%> (-1.90%) ⬇️
...gins/analysis/core/model/ResetReferenceAction.java 57.14% <ø> (ø)
...kins/plugins/analysis/core/model/ResultAction.java 66.66% <0.00%> (-1.34%) ⬇️
...gins/analysis/core/util/AffectedFilesResolver.java 91.83% <ø> (ø)
...enkins/plugins/analysis/core/util/QualityGate.java 50.48% <ø> (ø)
...lysis/warnings/axivion/RemoteAxivionDashboard.java 0.00% <0.00%> (ø)
...jenkins/plugins/analysis/core/model/JobAction.java 70.58% <7.69%> (-14.78%) ⬇️
... and 18 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 07d0a5f...383e208. Read the comment docs.

Copy link
Member

@simonsymhoven simonsymhoven left a comment

Choose a reason for hiding this comment

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

This looks very nice! Thanks.

@uhafner
Copy link
Member Author

uhafner commented Jul 10, 2021

@simonsymhoven one additional question: the texts "Quality Gate", "Success" and "No issues have been reported" (see screenshots above) seem to be very light when compared to all other texts in Jenkins. Is this intended or is there something broken in the CSS?

@simonsymhoven
Copy link
Member

@simonsymhoven one additional question: the texts "Quality Gate", "Success" and "No issues have been reported" (see screenshots above) seem to be very light when compared to all other texts in Jenkins. Is this intended or is there something broken in the CSS?

Hmm, at least I didn't consciously change the font settings.. I think there is something broken.

@uhafner uhafner merged commit 1df60e8 into master Jul 11, 2021
@uhafner uhafner deleted the status-icons-for-portlet branch July 11, 2021 08:25
@simonsymhoven
Copy link
Member

@uhafner When do you plan to release it? I have a presentation of the pull request monitoring plugin on wednesday, so it would be great if I could show the latest portlets. Thanks!

@uhafner
Copy link
Member Author

uhafner commented Jul 14, 2021

Hmm, last night I tried but the release process stopped locally due to some failures during Jenkins startup... I don't think that I find time today to find the reason.

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.

2 participants