Skip to content

Commit

Permalink
[#1619] BUILD: re-enable CI for all versions of java; build script cl…
Browse files Browse the repository at this point in the history
…eanup
  • Loading branch information
remkop committed Mar 2, 2022
1 parent 4a12a71 commit bc6e585
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 23 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ jobs:
build-java-8-plus:
strategy:
matrix:
java-version: [ 8 , 9 ] #, 10, 11, 12, 13, 14, 15, 16, 17, 18-ea ]
java-version: [ 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18-ea ]
os: [ubuntu-latest, macos-latest, windows-latest]

runs-on: ${{ matrix.os }}
Expand Down
11 changes: 0 additions & 11 deletions picocli-annotation-processing-tests/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -9,17 +9,6 @@ version "$projectVersion"
sourceCompatibility = 1.8
targetCompatibility = 1.8

//if (!JavaVersion.current().isJava11Compatible()) {
// sourceCompatibility = 1.8
// targetCompatibility = 1.8
//} else { // from Java 11 and up
// tasks.withType(JavaCompile).configureEach {
// javaCompiler = javaToolchains.compilerFor {
// languageVersion = JavaLanguageVersion.of(JavaVersion.current().majorVersion)
// }
// }
//}

dependencies {
api rootProject
api project(':picocli-codegen')
Expand Down
12 changes: 1 addition & 11 deletions picocli-codegen/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -9,17 +9,7 @@ ext.moduleName = 'info.picocli.codegen'

sourceCompatibility = 1.6
targetCompatibility = 1.6
//if (!org.gradle.api.JavaVersion.current().isJava11Compatible()) {
// sourceCompatibility = 1.6
// targetCompatibility = 1.6
//} else {
// // Java 13 cannot compile to target Java 6
// tasks.withType(JavaCompile).configureEach {
// javaCompiler = javaToolchains.compilerFor {
// languageVersion = JavaLanguageVersion.of(JavaVersion.current().majorVersion)
// }
// }
//}

dependencies {
api rootProject
}
Expand Down

0 comments on commit bc6e585

Please sign in to comment.