Skip to content

Commit

Permalink
Fix which JDKs are installed on CI (#835)
Browse files Browse the repository at this point in the history
We need some version of JDK 21 installed now (after #834) for the
snapshot job to succeed, even though it's not used. Take the opportunity
to simplify other usage of `setup-java`.
  • Loading branch information
msridhar committed Sep 28, 2023
1 parent d04828a commit 7e36bb0
Showing 1 changed file with 8 additions and 14 deletions.
22 changes: 8 additions & 14 deletions .github/workflows/continuous-integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,21 +36,13 @@ jobs:
steps:
- name: Check out NullAway sources
uses: actions/checkout@v3
- name: 'Set up JDK 21 so it is available'
- name: 'Set up JDKs'
uses: actions/setup-java@v3
with:
java-version: '21-ea'
distribution: 'temurin'
- name: 'Set up JDK 17 on Windows'
uses: actions/setup-java@v3
with:
java-version: '17'
distribution: 'temurin'
if: matrix.os == 'windows-latest'
- name: 'Set up JDK ${{ matrix.java }}'
uses: actions/setup-java@v3
with:
java-version: ${{ matrix.java }}
java-version: |
21-ea
17
${{ matrix.java }}
distribution: 'temurin'
- name: Build and test using Java ${{ matrix.java }} and Error Prone ${{ matrix.epVersion }}
env:
Expand Down Expand Up @@ -99,7 +91,9 @@ jobs:
- name: 'Set up JDK 11'
uses: actions/setup-java@v3
with:
java-version: 11
java-version: |
21-ea
11
distribution: 'temurin'
- name: 'Publish'
uses: gradle/gradle-build-action@v2
Expand Down

0 comments on commit 7e36bb0

Please sign in to comment.