search

galexandre / sonar-cobertura

Cobertura plugin for SonarQube
GNU General Public License v3.0
30 stars 26 forks source link

Compatibility with SQ 6.1 #9

Closed bguerin closed 7 years ago

bguerin commented 7 years ago

Fix #4

agaidaichuk commented 7 years ago

Hi, @bguerin . First of all, thanks a lot for so quick fix. But is there any ETA to get it merged?

bguerin commented 7 years ago

I am not the maintener of this plugin ... so cannot answer :(

ejocharcole commented 7 years ago

This fix works very well, I'm using it with my Sonar install. thank you! (I had an unusable Sonar 6.1 until I saw this PR)

arandre commented 7 years ago

Hi @ejocharcole and @bguerin

Thanks for getting this to work - just like @ejocharcole, Sonar 6.1 was dead in the water for me. Question - how did you install it for Sonar? Sorry for the question, but this is all new to me.

Thanks

ejocharcole commented 7 years ago

Checkout the branch and maven package. stop sonar then just copy the jar into extensions/plugins in your Sonar install - and remove the old version start sonar

then just check the plugins page in the console, you should see your new version of the plugin

if you've not updated your other plugins for a while you might find you need to update them - e.g. the Java plugin. I usually just download them and install them all myself in extensions/plugins (the Sonar console does have a feature for updating some plugins directly, but it didn't seem to work last time I tried it)

arandre commented 7 years ago

@ejocharcole - I checked out the branch but don't see the maven package.

ejocharcole commented 7 years ago

are you familiar with Maven? you need to run mvn package (i.e. in your IDE or command line)

arandre commented 7 years ago

yes - I got it right after i submitted the comment. Thanks once again!

ervinadr commented 7 years ago

Hi, We faced this problem as well. What is a plan when this is going to be merged? Thanks

benzonico commented 7 years ago

To answer the questions about ETA on merging this PR please carefully read the following : https://github.com/SonarQubeCommunity/sonar-cobertura#plugin-seeking-maintainers

This plugin is no longer maintained by Sonarsource, as a consequence in order for this PR to be merged someone as to step up from community to become maintainer.

If anyone is interested, please reach out to sonarqube@googlegroups.com

ejocharcole commented 7 years ago

thanks for the 6.2 fix @bguerin , works great.