-
Notifications
You must be signed in to change notification settings - Fork 2
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
SONARGO-52 Create a new repository for common Gradle scripts #33
Conversation
* Update Manifest and Badges * Fix Badge image
Update dependencies and migrate to version catalogs
[submodule "build-logic"] | ||
path = build-logic | ||
url = https://github.com/SonarSource/cloud-native-gradle-modules | ||
branch = SONARGO-52 |
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.
Temporary, until SonarSource/cloud-native-gradle-modules#1 is merged
java.sourceCompatibility = JavaVersion.VERSION_17 | ||
tasks.withType(JavaCompile) { | ||
options.encoding = "UTF-8" | ||
options.release = java.sourceCompatibility.majorVersion as int |
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.
Already default behavior
@@ -30,11 +31,6 @@ dependencies { | |||
testRuntimeOnly testLibs.junit.jupiter.engine | |||
} | |||
|
|||
tasks.withType(JavaCompile) { | |||
// Prevent warning: Gradle 5.0 will ignore annotation processors | |||
options.compilerArgs += [ "-proc:none" ] |
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 didn't see any warnings, maybe outdated already?
Quality Gate passed for 'Go'Issues Measures |
b186fe4
to
7e4becb
Compare
SONARGO-52