-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
sonar upgrade to 5.0.0 #5194
Conversation
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:
|
592d318
to
3311eba
Compare
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 |
making it draft, did not yet succeed in starting that copyInMissingTemplates task properly via dependency :( |
0e919ab
to
4bd3b98
Compare
4bd3b98
to
20b37b9
Compare
@soloturn are you planning to continue with this? |
no, not important any more, as sonar released a new version. updated to version 5 and dropped the rest of the change. |
20b37b9
to
ce8871a
Compare
There was a problem hiding this 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.
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