Add SonarQube analysis and coverage reporting to GitHub Actions#12550
Open
poikilotherm wants to merge 2 commits into
Open
Add SonarQube analysis and coverage reporting to GitHub Actions#12550poikilotherm wants to merge 2 commits into
poikilotherm wants to merge 2 commits into
Conversation
poikilotherm
added
Component: Code Infrastructure
|
Coverage is 24.974% — 9846-enable-sonarqube into develop. No base build found for |
…ions - Implemented SonarQube job in Maven CI workflow for enhanced code quality analysis. - Updated `pom.xml` and parent POM with SonarQube configuration properties and plugin definitions. - Refined JaCoCo configuration to support integrated coverage reporting.
poikilotherm
force-pushed
the
9846-enable-sonarqube
branch
from
July 21, 2026 17:22
1924deb to
d10c0dc
Compare
This comment has been minimized.
This comment has been minimized.
1 similar comment
This comment has been minimized.
This comment has been minimized.
- Adjusted artifact handling to include binary class files required by SonarQube. - Set `fetch-depth` to 0 for full Git history to improve Sonar analysis. - Updated JaCoCo XML report path in `pom.xml` for accurate coverage reporting (Sonar needs the full path to the XML file).
poikilotherm
force-pushed
the
9846-enable-sonarqube
branch
from
July 21, 2026 17:56
8ad43a2 to
3985900
Compare
This comment has been minimized.
This comment has been minimized.
|
|
📦 Pushed preview images as 🚢 See on GHCR. Use by referencing with full name as printed above, mind the registry name. |
Contributor
Author
|
I gave this a size 20 because I think before we merge this, we should:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it:
Enable better code quality by using SonarCloud.
Which issue(s) this PR closes:
Special notes for your reviewer:
None yet.
Suggestions on how to test this:
None yet.
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
Nope.
Is there a release notes update needed for this change?:
Not sure. Maybe?
Additional documentation:
None?