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

support 7.4 sonarqube (#1) #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Eli8838
Copy link

@Eli8838 Eli8838 commented Jan 10, 2019

No description provided.

TobiX referenced this pull request in Inform-Software/sonar-groovy Jan 15, 2019
CI build also runs against 7.3 and 7.5 to validate compatibility.

This closes #86 and fixes #72.
@NickLarsenNZ
Copy link

Let's go ;)

Is there anything stopping this being merged?

@deepak-kumbhar
Copy link

Hi,
Can you merge this PR? So we can use.

@Eli8838
Copy link
Author

Eli8838 commented Jun 6, 2019

only the owner of the original project can merge this PR.
but you can use the fork instead. that is what i am doing.
https://github.com/Eli8838/sonar-groovy

@deepak-kumbhar
Copy link

@Eli8838 Hi, I forked this PR but still, I am getting an error while scanning groovy project, Is there any specific way to scan the project? Or any process I need to follow. I forked the repository and build the project after that added the jar file on sonar's plugins.

@Eli8838
Copy link
Author

Eli8838 commented Jun 6, 2019

what kind of error are you getting?

@deepak-kumbhar
Copy link

This one, @Eli8838

11:25:37.912 INFO: Load metrics repository (done) | time=41ms 11:25:39.859 INFO: ------------------------------------------------------------------------ 11:25:39.861 INFO: EXECUTION FAILURE 11:25:39.862 INFO: ------------------------------------------------------------------------ 11:25:39.862 INFO: Total time: 6.465s 11:25:40.041 INFO: Final Memory: 45M/109M 11:25:40.042 INFO: ------------------------------------------------------------------------ 11:25:40.042 ERROR: Error during SonarQube Scanner execution 11:25:40.042 ERROR: org.sonar.plugins.groovy.codenarc.CodeNarcSensor has unsatisfied dependency 'class org.sonar.plugins.groovy.foundation.GroovyFileSystem' for constructor 'public org.sonar.plugins.groovy.codenarc.CodeNarcSensor(org.sonar.api.profiles.RulesProfile,org.sonar.plugins.groovy.foundation.GroovyFileSystem)' from org.sonar.core.platform.ComponentContainer$ExtendedDefaultPicoContainer@26935403:125<[Immutable]:org.sonar.core.platform.ComponentContainer$ExtendedDefaultPicoContainer@3eb82ab3:328<[Immutable]:org.sonar.core.platform.ComponentContainer$ExtendedDefaultPicoContainer@5f667fe2:55<| 11:25:40.042 ERROR: 11:25:40.042 ERROR: Re-run SonarQube Scanner using the -X switch to enable full debug logging.

@Eli8838
Copy link
Author

Eli8838 commented Jun 6, 2019

it seems that in sonarQube 7.7 they broke this plugin again. i will create a new fork for 7.7 and fix that there.

@deepak-kumbhar
Copy link

deepak-kumbhar commented Jun 6, 2019

Yes, thanks. That will be great :) Can you fix for both 7.6 and 7.7? Also please let me know once you fix. 💯

@alin-simionoiuDE
Copy link

any progress?

@Eli8838
Copy link
Author

Eli8838 commented Jun 19, 2019

Unfortunately the change required is too big for me to try and fix at this time...
the issue is something similar to:
jborgers/sonar-pmd#81

@TobiX
Copy link

TobiX commented Jul 11, 2019

This (and other issues) are fixed in release 1.6

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

Successfully merging this pull request may close these issues.

5 participants