-
Notifications
You must be signed in to change notification settings - Fork 6
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
Sonarcloud ci #5
Conversation
71a3e0d
to
1493894
Compare
e7762a4
to
a585888
Compare
7c8c7e1
to
79c6f54
Compare
PRs will be analyzed using SonarCloud with a CI based approach. Adds code coverage reporting using maven jacoco plugin. Updates maven.yaml to use java version based on pom.xml config.
79c6f54
to
a1626d4
Compare
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me (with a little help from Chat GPT to understand what you have done) :-)
I also tried to specify what changes and why in the first comment of the pullrequest. |
Yes, I know. :-) But I needed ChatGPT to understand lines like this: Java_Version=$(mvn help:evaluate "-Dexpression=maven.compiler.release" -q -DforceStdout | sed -e 's/^1./1.0./') :-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the update with autodetection instead of hardcoding which version should be used,
Also good with fetch.depth 0 for full history analysis
i see no problems with this, good work
Closes #4
Add more automatic checking of PR. The following changes are made:
maven.yml
Update maven.yml to detect java version from pom.xml setting maven.compiler.release instead of hard coding.
Check out more than top commit for better code analyzing. (fetch-depth: 0)
Add caching of downloaded sonar artifacts for faster runs in the future.
Add SonarCloud scanning of the code by adding sonar plugin to mvn and reports will be available in PR.
pom.xml
Add properties with settings for sonar.
Add code coverage report with jacoco plugin to pom.xml.