Skip to content

Commit

Permalink
Install Python earlier in CI
Browse files Browse the repository at this point in the history
  • Loading branch information
bglw committed Sep 29, 2024
1 parent 72595ca commit 8ad5f0e
Showing 1 changed file with 9 additions and 10 deletions.
19 changes: 9 additions & 10 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,13 @@ jobs:
default: true
components: rustfmt, clippy

- name: Set up python 3.12
uses: actions/setup-python@v5
with:
python-version: "3.12"
# NOTE: ^this strategy leaves older python versions intentionally
# without test coverage to keep CI fast.

- name: Check versions
run: |
cargo --version
Expand Down Expand Up @@ -112,16 +119,9 @@ jobs:
- name: Test CLI
run: ./test_ci.sh "release"

- name: Set up python 3.12
uses: actions/setup-python@v5
with:
python-version: "3.12"
# NOTE: ^this strategy leaves older python versions intentionally
# without test coverage to keep CI fast.

- name: Set up poetry
run: ./wrappers/python/scripts/ci/github/setup_poetry.sh

- name: cache venv
uses: actions/cache@v4
with:
Expand All @@ -142,7 +142,7 @@ jobs:
if: runner.os == 'Linux'
working-directory: ./wrappers/python
run: ./scripts/ci/python_lints.sh

- name: ensure cog up-to-date
# avoid duplicating linting work on different OSes
if: runner.os == 'Linux'
Expand All @@ -161,4 +161,3 @@ jobs:
# ^ guard against the tests getting deadlock if the subprocess pipe
# gets clogged
run: ./wrappers/python/scripts/ci/github/integration_tests.sh

0 comments on commit 8ad5f0e

Please sign in to comment.