From 4a7e14e5ad437cb8a768ba8f75cd12d3b2a16e6a Mon Sep 17 00:00:00 2001 From: Avasam Date: Mon, 2 Jan 2023 17:19:47 -0500 Subject: [PATCH] Deduplicate stubtest command in actions (#9444) --- .github/workflows/daily.yml | 5 +++-- .github/workflows/stubtest_third_party.yml | 6 ++++-- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/.github/workflows/daily.yml b/.github/workflows/daily.yml index 6dafad073..82a8b46b5 100644 --- a/.github/workflows/daily.yml +++ b/.github/workflows/daily.yml @@ -78,7 +78,7 @@ jobs: sudo apt update && sudo apt install -y $PACKAGES fi - xvfb-run python tests/stubtest_third_party.py --specified-stubs-only --num-shards 4 --shard-index ${{ matrix.shard-index }} + PYTHON_EXECUTABLE="xvfb-run python" else if [ "${{ runner.os }}" = "macOS" ] && [ -n "$PACKAGES" ]; then NONINTERACTIVE=1 brew install $PACKAGES @@ -88,9 +88,10 @@ jobs: choco install -y $PACKAGES fi - python tests/stubtest_third_party.py --specified-stubs-only --num-shards 4 --shard-index ${{ matrix.shard-index }} + PYTHON_EXECUTABLE="python" fi + $PYTHON_EXECUTABLE tests/stubtest_third_party.py --specified-stubs-only --num-shards 4 --shard-index ${{ matrix.shard-index }} stub-uploader: name: Run the stub_uploader tests runs-on: ubuntu-latest diff --git a/.github/workflows/stubtest_third_party.yml b/.github/workflows/stubtest_third_party.yml index 3e17e2f5b..a908d3695 100644 --- a/.github/workflows/stubtest_third_party.yml +++ b/.github/workflows/stubtest_third_party.yml @@ -66,7 +66,7 @@ jobs: sudo apt update && sudo apt install -y $PACKAGES fi - xvfb-run python tests/stubtest_third_party.py --specified-stubs-only $STUBS + PYTHON_EXECUTABLE="xvfb-run python" else if [ "${{ runner.os }}" = "macOS" ] && [ -n "$PACKAGES" ]; then echo "Installing Homebrew packages: $PACKAGES" @@ -78,8 +78,10 @@ jobs: choco install -y $PACKAGES fi - python tests/stubtest_third_party.py --specified-stubs-only $STUBS + PYTHON_EXECUTABLE="python" fi + + $PYTHON_EXECUTABLE tests/stubtest_third_party.py --specified-stubs-only $STUBS else echo "Nothing to test" fi