-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Can we get Sonarcloud setup for our main repository? #5577
Comments
I don't see why not ! it is a good idea ! |
Note that maintaining SonarQube is like a full-time position, due to the amount of false positives... Still a good idea, but none of the current complaints seems like it was actually any real issue ? i.e. it currently lists:
All of these would need to be handled. PS. I addressed the 3 Go bugs, the rest were in the website (8 HTML and 1 CSS) And of the vulnerabilities, it is all about having "hardcoded IP addresses" in the tests. |
Is this still something we're interested in supporting? |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Can we get Sonarcloud setup for our main repository? This is free for open source projects and provides valuable metrics like code quality issues and what not. It also supports Pull Request validations.
I have set it up on my fork and you can see the results over here - https://sonarcloud.io/dashboard?id=blueelvis_minikube
-Pranav
The text was updated successfully, but these errors were encountered: