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

Ensure correct coverage is reported to Sonarcloud #843

Merged
merged 1 commit into from
Dec 31, 2023

Conversation

mc1arke
Copy link
Owner

@mc1arke mc1arke commented Dec 31, 2023

Since the Gradle upgrade and the associated Sonarqube plugin upgrade, Sonarcloud has been reporting no coverage on the project's main branch. This is due to the Sonarqube plugin no longer executing the test task so have no coverage report available. The relevant pipeline steps are being altered to run the test task before the Sonarqube task, and have switched to the new sonar name for the task, as well as removing the compile task dependency since this is implicit from the test task and matches the future state of Sonarqube's Gradle plugin.

Since the Gradle upgrade and the associated Sonarqube plugin upgrade,
Sonarcloud has been reporting no coverage on the project's main branch.
This is due to the Sonarqube plugin no longer executing the test task
so have no coverage report available. The relevant pipeline steps are
being altered to run the `test` task before the Sonarqube task, and have
switched to the new `sonar` name for the task, as well as removing the
compile task dependency since this is implicit from the test task and
matches the future state of Sonarqube's Gradle plugin.
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@mc1arke mc1arke merged commit 45b667b into master Dec 31, 2023
12 checks passed
@mc1arke mc1arke deleted the sonar-gradle-execution branch December 31, 2023 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant