From 9a2bdbc34140309c6a71c8fc5f15c0e5097dca38 Mon Sep 17 00:00:00 2001 From: Adam Ling Date: Wed, 11 Oct 2023 23:19:42 -0700 Subject: [PATCH] fix precommit --- .github/workflows/precommit.yml | 28 ++++++++++++++++++++++++---- 1 file changed, 24 insertions(+), 4 deletions(-) diff --git a/.github/workflows/precommit.yml b/.github/workflows/precommit.yml index 945972a143a..be4757da275 100644 --- a/.github/workflows/precommit.yml +++ b/.github/workflows/precommit.yml @@ -101,7 +101,7 @@ jobs: run: python -m pip install -U setuptools pip wheel - name: Install tox run: python -m pip install tox - - if: ${{ contains('macos', matrix.os.download_name) }} + - if: ${{ not (matrix.python-version == '3.11' && matrix.cloud-provider == 'gcp') && contains('macos', matrix.os.download_name) }} name: Run doctests run: python -m tox -e "py${PYTHON_VERSION}-doctest-notudf-ci" env: @@ -112,7 +112,7 @@ jobs: # Specify SNOWFLAKE_IS_PYTHON_RUNTIME_TEST: 1 when adding >= python3.12 with no server-side support # For example, see https://github.com/snowflakedb/snowpark-python/pull/681 shell: bash - - if: ${{ matrix.python-version != '3.11' && matrix.cloud-provider != 'gcp' }} + - if: ${{ not (matrix.python-version == '3.11' && matrix.cloud-provider == 'gcp') }} name: Run tests (excluding doctests) run: python -m tox -e "py${PYTHON_VERSION/\./}-notdoctest-ci" env: @@ -121,6 +121,16 @@ jobs: PYTEST_ADDOPTS: --color=yes --tb=short TOX_PARALLEL_NO_SPINNER: 1 shell: bash + - if: ${{ matrix.python-version == '3.11' && matrix.cloud-provider == 'gcp' }} + name: Run tests (excluding udf and doctest tests) + run: python -m tox -e "py${PYTHON_VERSION/\./}-notudfdoctest" + env: + PYTHON_VERSION: ${{ matrix.python-version }} + cloud_provider: ${{ matrix.cloud-provider }} + PYTEST_ADDOPTS: --color=yes --tb=short + TOX_PARALLEL_NO_SPINNER: 1 + SNOWFLAKE_IS_PYTHON_RUNTIME_TEST: 1 + shell: bash - name: Combine coverages run: python -m tox -e coverage --skip-missing-interpreters false shell: bash @@ -212,7 +222,7 @@ jobs: run: python -m pip install -U setuptools pip wheel - name: Install tox run: python -m pip install tox - - if: ${{ contains('macos', matrix.os.download_name) }} + - if: ${{ not (matrix.python-version == '3.11' && matrix.cloud-provider == 'gcp') && contains('macos', matrix.os.download_name) }} name: Run doctests run: python -m tox -e "py${PYTHON_VERSION}-doctest-notudf-ci" env: @@ -221,7 +231,7 @@ jobs: PYTEST_ADDOPTS: --color=yes --tb=short --disable_sql_simplifier TOX_PARALLEL_NO_SPINNER: 1 shell: bash - - if: ${{ matrix.python-version != '3.11' && matrix.cloud-provider != 'gcp' }} + - if: ${{ not (matrix.python-version == '3.11' && matrix.cloud-provider == 'gcp') }} name: Run tests (excluding doctests) run: python -m tox -e "py${PYTHON_VERSION/\./}-notdoctest-ci" env: @@ -230,6 +240,16 @@ jobs: PYTEST_ADDOPTS: --color=yes --tb=short --disable_sql_simplifier TOX_PARALLEL_NO_SPINNER: 1 shell: bash + - if: ${{ matrix.python-version == '3.11' && matrix.cloud-provider == 'gcp' }} + name: Run tests (excluding udf and doctest tests) + run: python -m tox -e "py${PYTHON_VERSION/\./}-notudfdoctest" + env: + PYTHON_VERSION: ${{ matrix.python-version }} + cloud_provider: ${{ matrix.cloud-provider }} + PYTEST_ADDOPTS: --color=yes --tb=short --disable_sql_simplifier + TOX_PARALLEL_NO_SPINNER: 1 + SNOWFLAKE_IS_PYTHON_RUNTIME_TEST: 1 + shell: bash - name: Combine coverages run: python -m tox -e coverage --skip-missing-interpreters false shell: bash