Continuous integration improvements#2789
Continuous integration improvements#2789jord1e wants to merge 49 commits intographql-java:masterfrom
Conversation
Fixed: - Generic warnings introduced in Java 9 when using `Stream#of` is used in combination with `Function.identity()`, probably caused by JEP 215 "Tiered Attribution for javac" - Java 13 introduced `String#stripIndent`, which conflicted with Groovy itself, see apache/groovy#1139, the errors were probably caused by whitespace indentation which was handled differently - Resolved compiler warning about missing `@Deprecated` on deprecated `ExecutionStepInfo#getFieldContainer` method (JDK 9/JEP 277 "Enhanced Deprecation") - The wrong `BigInteger` constructor was being used in `ScalarIntTest.groovy` and `ScalarFloatTest.groovy` Not yet fixed: - Java 16 modified LineNumberReader#getLineNumber behavior (see JDK-8241020) which messed up MultiSourceReader Links: - apache/groovy#1139 - https://bugs.openjdk.java.net/browse/JDK-8241020 - https://github.com/openjdk/jdk/commits/master/src/java.base/share/classes/java/io/LineNumberReader.java
|
Hey @jord1e is there anything else you want to add before review? |
No. Go ahead :) |
# Conflicts: # .github/workflows/pull_request.yml # build.gradle
|
@dondonz I updated the branch |
.github/workflows/pull_request.yml
Outdated
| - { c: '8.0.282', t: '8.0.282' } # Earliest supported Java 8 | ||
| - { c: '11', t: '11' } | ||
| - { c: '17', t: '17' } | ||
| - { c: '18', t: '18' } |
There was a problem hiding this comment.
Given that 18 EOL was reached 1 year ago, we should be able to drop this and replace it with 17, 21, and possibly 22
There was a problem hiding this comment.
now running 11,17,21
I think adding 22-ea in a new PR makes the most sense
Temurin is already releasing EA builds, we just need to make sure Semeru is excluded from the test matrix (or have includes: on 22-ea)
# Conflicts: # .github/workflows/master.yml # .github/workflows/pull_request.yml # .github/workflows/release.yml # settings.gradle
[skip ci]
| - name: Build | ||
| uses: gradle/gradle-build-action@v2 | ||
| with: | ||
| arguments: assemble | ||
| - name: Test | ||
| uses: gradle/gradle-build-action@v2 | ||
| with: | ||
| arguments: check --info | ||
| - name: Publish | ||
| uses: gradle/gradle-build-action@v2 | ||
| with: | ||
| arguments: publishToSonatype closeAndReleaseSonatypeStagingRepository -x check --info --stacktrace |
There was a problem hiding this comment.
Facilitates better caching
| - name: Build | ||
| uses: gradle/gradle-build-action@v2 | ||
| with: | ||
| arguments: assemble | ||
| - name: Test | ||
| uses: gradle/gradle-build-action@v2 | ||
| with: | ||
| arguments: check --info | ||
| - name: Publish | ||
| uses: gradle/gradle-build-action@v2 | ||
| with: | ||
| arguments: publishToSonatype closeAndReleaseSonatypeStagingRepository -x check --info --stacktrace |
There was a problem hiding this comment.
Facilitates better caching
| plugins { | ||
| id 'org.gradle.toolchains.foojay-resolver-convention' version '0.7.0' | ||
| } | ||
|
|
There was a problem hiding this comment.
Has to be defined in the settings.gradle
Makes sure the relevant JDK is automatically downloaded again. This was changed in a more recent version (of Gradle), it used to download the JDK automatically if it was missing--now we need this plugin
|
Tests >= 17 broken because of #3369 |
|
Hello coming back after a long time to say thanks again for your PR. We remixed some of your ideas and introduced new GitHub actions #4002 |
3 participants
Fixes #2676
Does the following:
sourceCompatibilityandtargetCompatibility(now resides in gradle.properties and just takes whatever J8 you have installed, or installs the latest J8)Does not touch deployment of the jars on master, I found it kind of unnecessary because everything is done in PRs anyway and it complicates the build scripts.
This needs to be squashed before merging.