-
-
Notifications
You must be signed in to change notification settings - Fork 528
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
Enhancement: Sonarqube 10.4 support #870
Comments
What is an error with 10.4? |
It’s not yet listed in readme for this repo thus I haven’t even tried upgrading SQ (and plugin) |
Then why have you labeled it as bug? :) |
@nixel2007 because while submitting from mobile phone I haven't managed to change label to enhancement (that's why I added it in title as prefix) 🤷 |
I've not attempted a build of the plugin against the latest Sonarqube release yet. I think the plugin will work with Sonarqube 10.4, but the tests will probably fail to compile. I'll try and run my test pack this coming week and confirm. |
Thank you. |
I see the following error after update when I am validating GitHub integration:
|
I was able to make the plugin work with SonarQube 10.4 a few minor code changes. |
Just tried the latest version with 10.4-community. |
Would be great to get a version for 10.4.1 support also |
The plugin seems to run with SQ 10.4.1 (scans work, and the results land in the right branch), but the new Overview dashboard seems to have a few broken links:
|
@cbaerikebc when you say 'The plugin',which one are you referring to: mc1arke or GreyTeardrop's fork? |
@mukundshah this is with @mc1arke's plugin, version 1.18.0 |
Updates to the latest Sonarqube minor version, including changes to the Github decorator to handle changes in the JTW library method signatures inherited from Sonarqube core, and test only changes to the Class Loader code built into Sonarqube.
Updates to the latest Sonarqube minor version, including changes to the Github decorator to handle changes in the JTW library method signatures inherited from Sonarqube core, and test only changes to the Class Loader code built into Sonarqube.
This has not been included in the plugin yet. I'd held up the release whilst I'd tried to get something that worked across all the decorators, but ultimately wasn't comfortable that the additional changes were ready. |
Released in 1.19.0 of the plugin |
Hi @mc1arke!
First of all I would take a chance to thank you for this plugin which allows to use SQ premium benefits for free SQ version, your huge efforts are appreciated 🔥
SQ 10.4 was recently released, so creating this ticket to know if the support for it will be added 🙏🏻
The text was updated successfully, but these errors were encountered: