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

[question] Support for SonarQube 5.6 #30

Open
tristanlins opened this issue Jun 14, 2016 · 12 comments
Open

[question] Support for SonarQube 5.6 #30

tristanlins opened this issue Jun 14, 2016 · 12 comments

Comments

@tristanlins
Copy link

SonarQube 5.6 LTS is released some days ago. Are there any plans / timeline to support the new version?

@rmalveis
Copy link

+1

@aurelien-baudet
Copy link
Member

Sonar has made big changes on plugin APIs and some APIs are no longer available or working as before. For example, code duplication API is no more available. Now, if you want to support a language you have to provide a lexer based on SSLR. If you don't do that, you can't set information about the file metrics (number of lines, number of line of comments, number of line of code...).
I think that our plugin could be adapted quickly (code is quite simple and just simplifies calls to Sonar APIs) but the most difficult part is to understand and find a way to make features still working correctly with Sonar 5.6+.

For now, I tried to find solutions to make it work with Sonar 5.5. I didn't try for Sonar 5.6. We will try it but I don't know when and I can't ensure that it will be possible.

But it is an open source project so you could also try yourself and I will be pleased to have external contributors.

@mattadamson
Copy link

Hi @aurelien-baudet I'm very excited to see this plug in and also a little disappointed it won't work with 5.6. We are also looking to move to Sonar 5.6 LTS. On reviewing 5.5 did you find some of the standard rule violations will work correctly e.g. for eslint? I wonder if we can at least support a subset of the analysis and allow the others to be disabled whilst we work on getting the other report types imported.

Thoughts?

@mattadamson
Copy link

I noticed most of the reports are configurable so perhaps it does work in 5.6 for importing some linters such as eslint reports if others are turned off which don't work.

@merjadok
Copy link

merjadok commented Nov 9, 2016

Hello,

thank you for the great plugin!

After upgrading to Sonar version 5.6 we have experienced some problems so I wanted to check whether it is still the case that 5.6 version is not supported?

Also, we are now having a multi-module project: is this supported by the plugin?

Thanks in advance!

@NikitaEgorov
Copy link

@fsmaia
Copy link

fsmaia commented Nov 23, 2016

Hi @aurelien-baudet! Thanks for explanation about Sonar 5.6.

I'm running Sonar 5.5 with SonarScanner 2.8 (by [email protected]) and generating reports with sonar-web-frontend-reporters and JSCPD with [email protected].

However the duplication reports aren't reaching Sonar.

Do you know if this can be related to code duplication API deprecation?

Thanks in advance!

@noomidian
Copy link

@fsmaia @aurelien-baudet already answered this question in a previous post (see above)
It seems that he is pretty muched occupied this days.
About the problem of duplications, we have the same problem, we generate the js-duplication.xml with the sii reporter, the file is read and added to sonar api measures but the measures do not displays in the sonar interface (btw, we use the 5.6 release). I tried to debug quickly but i didn't notice something strange.

Btw, @aurelien-baudet, i did a pull request to correct a problem of regexp we encountered with the java plugin and Sonar 5.6 (the code resolved our problem).

We also did corrections in the reporters, i'll try to do a pull request for it this week.

@cedric-legallo
Copy link
Contributor

cedric-legallo commented Nov 28, 2016

We also did corrections in the reporters, i'll try to do a pull request for it this

week.

I have a bit more spare time than Aurelien has, so this one should be merged quickly. One day i'll dig in the plugin code enough to review and merge PR but i neither do have enough time for that ATM, so you'll have to wait for Aurelien to do it.

@rusboy
Copy link

rusboy commented Jan 19, 2017

+100500

@rusboy
Copy link

rusboy commented Jan 23, 2017

Hey Guys,
Can you explain why your excluded importing for native eslint rules from sonar-5.x branches (I'm observing only angular-eslint)?

Your tool looks good, however I need it with SonarQube 5.6....

I have an idea, to make the plugin works with SonarQube 5.6.
Will be useful, if you share your experience and found issues

Regards,
Ruslan

@CedricLeGallo
Copy link

Native eslint rules should be included. You have to configure your Sonar Rules to activate or deactivate ruleset. If the defaultruleset exclude them you can tune them in sonar.

The sure thing is the reporters report them. The main issue with upgrading the plugin for more recent sonar version is the pure removal by sonar of some API we used and don't have alternative anymore to display this informations.

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

No branches or pull requests