gabrie-allaigre / sonar-gitlab-plugin

Add to each commit GitLab in a global commentary on the new anomalies added by this commit and add comment lines of modified files
GNU Lesser General Public License v3.0
713 stars 208 forks source link

SonarQube failed to complete the review of this commit: No enum constant com.talanlabs.sonar.plugins.gitlab.models.QualityGate.Status.NONE #107

Closed demeureg closed 6 years ago

demeureg commented 6 years ago

Hello,

when using the latest version of this plugin, i've got this error :

SonarQube failed to complete the review of this commit: No enum constant com.talanlabs.sonar.plugins.gitlab.models.QualityGate.Status.NONE

Environment :

I remain at your disposal should you have any questions.

Guillaume Demeure

gabrie-allaigre commented 6 years ago

Hi, What is a command line ? Have you a complet log ? Thanks

gabrie-allaigre commented 6 years ago

You use quality gate info ? You have a quality gate status return NONE. I will correct the problem

demeureg commented 6 years ago

Sorry for the logs, i dont have it, but the command used with both version of maven plugin was :

mvn -Dsonar.branch.name=${GIT_BRANCH} -Dsonar.gitlab.commit_sha=${GIT_COMMIT} -Dsonar.gitlab.ref_name=${GIT_BRANCH} -Dsonar.host.url=${SONAR_URL} -Dsonar.gitlab.ignore_certificate=true -Dsonar.gitlab.project_id=${PROJECT_ID} -Dsonar.gitlab.user_token=${GITLAB_USER_TOKEN} -Dsonar.gitlab.all_issues=true -f app/pom.xml ${env.SONAR_MAVEN_GOAL}

gabrie-allaigre commented 6 years ago

Hi,

Can you test with https://github.com/gabrie-allaigre/sonar-gitlab-plugin/releases/download/3.0.1-rc1/sonar-gitlab-plugin-3.0.1-rc1.jar ?

Thanks

demeureg commented 6 years ago

Hello,

Thanks for the fix.

I guess it should solve the problem, i'll try to test today.

gabrie-allaigre commented 6 years ago

Ok, thanks

demeureg commented 6 years ago

Hello Gabrie,

i've found a workaround before your fix and i can't rollback projects to quality gate NONE (or i didn't found how).

I close the issue based on your fix, everything sounds good after reading the code.

Thanks for your support.