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

Configure Sonarcloud #1207

Open
JoerivanEngelen opened this issue Sep 10, 2024 · 0 comments
Open

Configure Sonarcloud #1207

JoerivanEngelen opened this issue Sep 10, 2024 · 0 comments
Assignees

Comments

@JoerivanEngelen
Copy link
Contributor

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/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 📝Refined
Development

No branches or pull requests

2 participants