-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Provide integration with Sonar #228
Comments
Just FYI, that project was not implemented by the gometalinter team, it's entirely third party. |
@alecthomas Is this implementation on the roadmap or no one will do it ? |
In this project? I have no idea, you'll need to ask @jirfag . |
Hi @jirfag, would u consider to implement an integration with Sonar ? |
hi! |
Actually it would be the other way around, SonarGo would need to support golangci-lint (ie by consuming the output and mapping to the rules/categories within Sonar). I created an issue with SonarGo for the same, https://github.com/SonarSource/sonar-go/issues/346. |
@kenjones-cisco Gometalinter has |
Resolving as there are now several output formats available that should work with various CI and reporting tools.
|
It would be good to have an integration with Sonar, the same as gometalinter has.
Please see https://github.com/uartois/sonar-golang
Thanks for really the great tool golangci-lint
The text was updated successfully, but these errors were encountered: