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

sonar upgrade to 5.0.0 #5194

Merged
merged 1 commit into from May 4, 2024
Merged

sonar upgrade to 5.0.0 #5194

merged 1 commit into from May 4, 2024

Conversation

soloturn
Copy link
Contributor

@soloturn soloturn commented Dec 9, 2023

the new default behavior of sonarscanner is to not compile any more, to permit easier re-run. it can be controlled by a flag. the default behavior changes from sonar 4.4.1 to sonar 5.0.0, as well the name of the flag is unlucky. see here for more explanation: https://community.sonarsource.com/t/sonarscanner-for-gradle-you-can-now-decide-when-to-compile

@github-actions github-actions bot added the Category: Build/CI Requests, Issues and Changes targeting gradle, groovy, Jenkins, etc. label Dec 9, 2023
@soloturn
Copy link
Contributor Author

soloturn commented Dec 9, 2023

this change does not work on jenkins. local, on root of the source code tree there was a gradle.properties, which is git-ignored. i deleted it, and it did not get recreated out of templates. how it is supposed to work to place it there and make it effective? i saw a task which does it on demand:

gradle copyInMissingTemplates

@BenjaminAmos
Copy link
Contributor

According to https://community.sonarsource.com/t/sonar-gradle-skipcompile-is-not-working/102710/9, that setting may not exist for much longer. I think a proper solution would involve configuring a compile dependency in build-logic/src/main/kotlin/terasology-metrics.gradle.kts, as we do already for the extractConfig gradle task.

@soloturn soloturn marked this pull request as draft December 22, 2023 08:41
@soloturn
Copy link
Contributor Author

soloturn commented Dec 22, 2023

making it draft, did not yet succeed in starting that copyInMissingTemplates task properly via dependency :(

@jdrueckert
Copy link
Member

@soloturn are you planning to continue with this?

@jdrueckert jdrueckert added the Status: Needs Author Input Requires more information by the author on the reported issue or provided changes label Apr 21, 2024
@soloturn
Copy link
Contributor Author

no, not important any more, as sonar released a new version. updated to version 5 and dropped the rest of the change.

@BenjaminAmos BenjaminAmos self-requested a review April 29, 2024 17:26
@BenjaminAmos BenjaminAmos marked this pull request as ready for review April 29, 2024 17:26
@BenjaminAmos BenjaminAmos changed the title sonar permits to decide when to compile sonar upgrade to 5.0.0 Apr 29, 2024
Copy link
Contributor

@BenjaminAmos BenjaminAmos left a comment

Choose a reason for hiding this comment

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

I have struggled to test this, since there does not appear to be any instructions available on running the SonarQube scanner on this project. It does not appear to be run in CI either. Still, if this fixes some verbose un-needed warnings during builds then it should be harmless but beneficial to merge.

@BenjaminAmos BenjaminAmos merged commit 072d207 into develop May 4, 2024
10 checks passed
@BenjaminAmos BenjaminAmos deleted the qa/future-sonar branch May 4, 2024 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Build/CI Requests, Issues and Changes targeting gradle, groovy, Jenkins, etc. Status: Needs Author Input Requires more information by the author on the reported issue or provided changes
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

3 participants