cathive / concourse-sonarqube-resource

performs SonarQube analyses and checks quality gates https://concourse-ci.org/ https://sonarqube.org/
Apache License 2.0
46 stars 51 forks source link

Missing Maven wrapper support #86

Open gdupontf opened 1 year ago

gdupontf commented 1 year ago

Title says all. It would be much appreciated if Maven wrapper support were to be added. At the minimum, another scanner option, mvnwwould do this. In addition, some minimal autodetection work to make this transparent would be even better.

headcr4sh commented 1 year ago

Another "mvnw" target seems a bit counter-unintuitive, I think. Auto-detection of a mvnw script on the other hand makes sense to me and might be implemented without too much effort.

busabuzz commented 1 year ago

Would be greatly appreciated, not able to get the resource to work with the maven wrapper currently.

holgerstolzenberg commented 8 months ago

I am doing some maintenance on the resource at the moment, but I have to admit I have no clue where in the usage of the resource the maven wrapper comes into play.

Can you please elaborate a bit more in detail, what you wanna achieve!?

gdupontf commented 8 months ago

It would be to use the maven wrapper script at the repository root if one is found. Ie the typical mvnw script. I would recommend to allow disabling autodetection as there's always the odd case.