Skip to content

Commit

Permalink
CI: make gradle-wrapper-validation a prereq only in gradle tests.
Browse files Browse the repository at this point in the history
Previously the gradle-wrapper-validation task was a prerequisite for
the build and test actions, which doesn't make sense since those actions
do not use gradle.

Instead, make gradle-wrapper-validation a prerequisite only for the
gradle-emulator-tests action. This should speed up CI latency by a few seconds,
since now gradle-wrapper-validation can run in parallel.
  • Loading branch information
brettchabot committed Jan 25, 2024
1 parent 7e6c095 commit 5563298
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ jobs:
- uses: gradle/wrapper-validation-action@v1
build:
runs-on: ubuntu-latest
needs: gradle-wrapper-validation
timeout-minutes: 20
steps:
- name: Check out repository code
Expand Down Expand Up @@ -64,7 +63,6 @@ jobs:
shell: bash
test:
runs-on: ubuntu-latest
needs: gradle-wrapper-validation
timeout-minutes: 20
steps:
- name: Check out repository code
Expand Down Expand Up @@ -96,7 +94,7 @@ jobs:
shell: bash
gradle-emulator-test:
runs-on: ubuntu-latest
needs: build
needs: [build, gradle-wrapper-validation]
timeout-minutes: 20
steps:
- name: Check out repository code
Expand Down

0 comments on commit 5563298

Please sign in to comment.