Skip to content

Commit

Permalink
Use Zulu JDK instead of Adopt for CI
Browse files Browse the repository at this point in the history
The new GitHub macOS host uses ARM, and the Adopt/Temurin JDKs no longer work since they aren't built for that architecture until Java 11.

See: actions/setup-java#625
  • Loading branch information
BenWoodworth committed May 7, 2024
1 parent 9ae8c57 commit 6580e06
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/publish.yml
Expand Up @@ -23,7 +23,7 @@ jobs:
uses: actions/setup-java@v4
with:
java-version: '8'
distribution: 'adopt'
distribution: 'zulu'
- name: Clean
if: ${{ false }} # Skip, since it fails on Windows: https://youtrack.jetbrains.com/issue/KT-50545/
run: .\gradlew.bat clean
Expand All @@ -46,7 +46,7 @@ jobs:
uses: actions/setup-java@v4
with:
java-version: '8'
distribution: 'adopt'
distribution: 'zulu'
- name: Clean
run: ./gradlew clean
- name: Publish to Maven
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/test.yml
Expand Up @@ -12,7 +12,7 @@ jobs:
uses: actions/setup-java@v4
with:
java-version: '8'
distribution: 'adopt'
distribution: 'zulu'
- name: Install ncurses5
run: sudo apt-get install libncurses5
- name: Clean
Expand All @@ -30,7 +30,7 @@ jobs:
uses: actions/setup-java@v4
with:
java-version: '8'
distribution: 'adopt'
distribution: 'zulu'
- name: Clean
if: ${{ false }} # Skip, since it fails on Windows: https://youtrack.jetbrains.com/issue/KT-50545/
run: .\gradlew.bat clean
Expand All @@ -46,7 +46,7 @@ jobs:
uses: actions/setup-java@v4
with:
java-version: '8'
distribution: 'adopt'
distribution: 'zulu'
- name: Clean
run: ./gradlew clean
- name: Test
Expand Down

0 comments on commit 6580e06

Please sign in to comment.