[NEW RELEASE] Clover plugin 4.0

Description: This new version brings compatibility with newer versions of SonarQube.

Compatibility: It have been tested on version 6.7+

https://sonarcloud.io/dashboard?id=sfeir-open-source_sonar-clover

Hi,

Two things. First, I see that you currently pass your quality gate, but it looks like that’s just because the new code aged-out of the leak period, and before that happened you would have failed for having 0% coverage? At the same time, I see that you do have test files in your project. Is something preventing you from feeding the coverage reports into analysis?

Second, as described in the docs, I need a PR for this on https://github.com/SonarSource/sonar-update-center-properties

 
Ann

Hi,

We had to rewrite a part of the build pipeline in order to build the project without https://github.com/SonarSource/travis-utils cause we’re not part of SonarSource and it has been difficult to find out some documentation for this tool, also it were simpler to create a brand new pipeline.
Writing this new pipeline, I thought that it was a bit confusing to have a plugin that’s help to push coverage data in SonarQube that have to use another coverage tools like jacoco :stuck_out_tongue:

Despite of this, I can easily understand that to be referenced in the market place, we have to follow some standards so if you think that’s it could be mandatory or just a nice to have thing, I can had the jacoco support in this repository build. Just let me know

You can find the PR in sonar-uprage-center-properties here : https://github.com/SonarSource/sonar-update-center-properties/pull/42 It was included in the first message (I were also confuse by the display format of the message :smile: )

Tony

Hi Tony,

Okay, yeah. It makes sense now why you’re not reporting coverage to SonarCloud. :roll_eyes:

Unfortunately, I think I do need to ask you to use JaCoCo for your SC analysis. :neutral_face:

Sorry I missed the link to your PR initially. Honestly the “one box” feature made it look like an ad and a little selective blindness kicked in. :joy:

I’ve reviewed the PR and requested some changes.

 
Ann

Hi Ann,

I’ve updated the project to include the coverage report. Should I publish a new version ? I mean the plugin code base is the same, but the commit used to publish the version is not the one that contains the coverage analysis in SonarCloud. Even if I’ve only updated the pipeline to check the coverage report, I’m not really comfortable with this situation

Also, I’ve updated the PR in sonar-update-center-properties to include the requested changes.

Tony

Hi Tony,

Sorry for the delay in responding. Your PR looks but something went sideways in your packaging; your manifest is nearly empty and it’s causing errors in the Marketplace metadata generator. I had merged your PR, but I’m going to have to back that out until your jar is fixed; it blocks the job.

 
Ann

Hi Ann,

I’ve tried to create a cleanest release. I hope this time it won’t cause any trouble.
You can find it here https://github.com/SonarSource/sonar-update-center-properties/pull/54
Also, is there a way to test this integration locally ?

Have a nice day,
Tony

Hi Tony,

The link in your PR goes to your sources zip, not the plugin itself.

Sorry, but there’s not a way to test locally.

 
Ann

Hi Ann,

Thank you for your review, it sees me that I also pointed the wrong version. It’s now fixed.

I can understand that there is no easy way to test this kind of things, so I hope that this time this contribution will not break anything :crossed_fingers:

Have a nice day,

Tony

Success!

 
:smiley:
Ann

1 Like

Thank you ! it’s awesome to finally see this new version appear in the marketplace :smile:
You make my day, hope you’ll have a nice day too :smiley:

Tony