-
Notifications
You must be signed in to change notification settings - Fork 3
Configure Sonarcloud #1207
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
Comments
My attempt was ignored by sonarcloud |
Fixed in #1440 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now, the Sonarcloud check is so strict it always raises an error. Most frustrating are its complaints about duplicate code in tests, where duplicate code quite often is a necessity or not worth the hassle to avoid; for example testcase strings for projectfiles.
I noticed myself just flat out ignoring it and just merging straightaway. This now gives the false impression to users viewing Github that we are merging broken branches. Sonarcloud therefore needs to be configured in some way. This cannot be done in the UI, but it should be possible with some settings files.
For example we could reduce the scope to ignore the tests folder, and the Kotlin scripts with parameters:
https://docs.sonarsource.com/sonarcloud/advanced-setup/analysis-scope/
Parameters cannot be set in the UI. Documentation on how to set parameters:
https://docs.sonarsource.com/sonarcloud/advanced-setup/analysis-parameters/
The text was updated successfully, but these errors were encountered: