From 6a238d0185d5b75b5570ec59bc848ec555a24eca Mon Sep 17 00:00:00 2001 From: tazlin Date: Mon, 15 Jul 2024 21:59:46 -0400 Subject: [PATCH] tests: fix pr test pre-commit runs; only install tox on runner --- .github/workflows/maintests.yml | 2 +- .github/workflows/prtests.yml | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/.github/workflows/maintests.yml b/.github/workflows/maintests.yml index 00bfe59c..a4b5c16f 100644 --- a/.github/workflows/maintests.yml +++ b/.github/workflows/maintests.yml @@ -40,7 +40,7 @@ jobs: - name: Install tox and any other packages run: | python -m pip install --upgrade pip - pip install --upgrade -r requirements.dev.txt + pip install --upgrade tox - name: Check build_helper.py horde-engine imports have no breaking dependency changes run: tox -e test-build-helper - name: Build unit test environment, confirm CUDA is available on host diff --git a/.github/workflows/prtests.yml b/.github/workflows/prtests.yml index 84f53298..91c56c36 100644 --- a/.github/workflows/prtests.yml +++ b/.github/workflows/prtests.yml @@ -19,6 +19,8 @@ jobs: runs-on: ubuntu-latest steps: - uses: actions/checkout@v3 + with: + ref: ${{ github.event.pull_request.head.sha }} - name: Run pre-commit uses: pre-commit/action@v3.0.0 with: @@ -45,7 +47,7 @@ jobs: - name: Install tox and any other packages run: | python -m pip install --upgrade pip - pip install --upgrade -r requirements.dev.txt + pip install --upgrade tox - name: Check build_helper.py horde-engine imports have no breaking dependency changes run: tox -e test-build-helper - name: Build unit test environment, confirm CUDA is available on host