-
-
Notifications
You must be signed in to change notification settings - Fork 530
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
10.7 upgrade #976
Comments
Does the plugin work properly on 10.7? |
Yes it works. Is more about releasing new docker image https://hub.docker.com/r/mc1arke/sonarqube-with-community-branch-plugin I know we can use official sonarqube docker image but this plugin image is really simpler to use |
Well, it works, but the report key for Code Insights has changed from the project key to "com.sonarsource.sonarqube"! So if you had previously defined a Code Insights report using the project key, you need to change it to the new key. |
This was called out in the release note (copied below), and isn't relevant to whether the plugin works on 10.7, given the same bahviour is exhibited on 10.6:
|
Okay, thanks for the clarification. We updated from Sonar 10.6. to 10.7 and at the same time updated the plugin from 1.21 to 1.22 and did not correctly assess the impact of this change. |
Hello @mc1arke Could you please check and confirm plugin version 1.22.0 is supported (working properly)for Sonarqube version 10.7 and update version matrix on the readme |
I am using 1.22.0 + 10.7 fine for our platform. README need updates ofc. |
I m using 1.22.0 + 10.7 with GitLab MR decoration without problems. |
Trying to accomplish this with Github Looks like github api has a typo
looks like my issue is related to #969 |
This will be fixed with a new release: #992 |
10.6 Sonarqube version has following bug https://sonarsource.atlassian.net/browse/SONAR-22149 that prevent displaying image from Sonaqube server
Resulting
Can we release new version of plugin for 10.7?
I've tried on my fork, 10.7 is just bump number version, no breaking change on API and other.
That why I didn't propose PR because is it will more about releasing process rather than coding value.
PS: with 10.7 version icon from report is correctly fixed
The text was updated successfully, but these errors were encountered: