You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
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: