-
Notifications
You must be signed in to change notification settings - Fork 481
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
Set up Sonar #9846
Comments
These properties were required to run this: export SONAR_TOKEN=xxxx mvn verify org.sonarsource.scanner.maven:sonar-maven-plugin:sonar -Dsonar.projectKey=IQSS_dataverse
See also (linking in the other direction)
I wonder what Sonar is going to be used for. There's a checkstyle workflow (that could be updated to highlight more issues) and similar actions for other tools exist. (I shouldn't get involved... I'm just happy that old issues are not completely forgotten.) |
Hey, @bencomp, thanks for stopping by. 😄 I just upgraded Netbeans from 14 to 18 and it was surprisingly easy to set up SolarLint this time. (At least it seems to be going well.) The plugin was easy to find this time: I tried this "analyze" button... ... and stuff started happening immediately: Maybe this is because we already set up https://sonarcloud.io/project/overview?id=IQSS_dataverse earlier. 🤔 I'm sort of surprised it's already analyzing and doing stuff. I assumed I'd have to enter some credentials for my new sonarcloud.io account or set up SonarCube server. 🤔 |
As I've mentioned before - the connection is necessary to download and apply customized rulesets. Either disabling / reconfiguring or adding new rules. Otherwise it will use the standard ruleset to give you at least something. |
I just found this: I'm not sure how it works with IntelliJ but I was assuming I'd give Netbeans https://sonarcloud.io/project/overview?id=IQSS_dataverse as a URL and automatically the rules would be downloaded. Maybe I can download and apply them manually? Shouldn't the rule be checked into our code base anyway? Here are the numbers we saw on solarcloud yesterday: Here are the numbers Netbeans is reporting: So yeah, it might be nice to get them to line up. |
These properties were required to run this: export SONAR_TOKEN=xxxx mvn verify org.sonarsource.scanner.maven:sonar-maven-plugin:sonar -Dsonar.projectKey=IQSS_dataverse
No description provided.
The text was updated successfully, but these errors were encountered: