Skip to content

Commit

Permalink
CI: Test various Pyarrow version on Linux (pandas-dev#46059)
Browse files Browse the repository at this point in the history
* CI: Test various Pyarrow version on Linux

* Ignore 3 and use conda

* use conda install

* fix concurrency group

* Don't update misc dependencies

* Adjust gbq test

* expand on comment
  • Loading branch information
mroeschke authored Feb 21, 2022
1 parent afc06e7 commit ea0184e
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
9 changes: 8 additions & 1 deletion .github/workflows/posix.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,9 @@ jobs:
matrix:
env_file: [actions-38.yaml, actions-39.yaml, actions-310.yaml]
pattern: ["not single_cpu", "single_cpu"]
# Don't test pyarrow v2/3: Causes timeouts in read_csv engine
# even if tests are skipped/xfailed
pyarrow_version: ["5", "6", "7"]
include:
- env_file: actions-38-downstream_compat.yaml
pattern: "not slow and not network and not single_cpu"
Expand Down Expand Up @@ -65,7 +68,7 @@ jobs:
COVERAGE: ${{ !contains(matrix.env_file, 'pypy') }}
concurrency:
# https://github.community/t/concurrecy-not-work-for-push/183068/7
group: ${{ github.event_name == 'push' && github.run_number || github.ref }}-${{ matrix.env_file }}-${{ matrix.pattern }}-${{ matrix.extra_apt || '' }}
group: ${{ github.event_name == 'push' && github.run_number || github.ref }}-${{ matrix.env_file }}-${{ matrix.pattern }}-${{ matrix.pyarrow_version || '' }}-${{ matrix.extra_apt || '' }}
cancel-in-progress: true

services:
Expand Down Expand Up @@ -133,6 +136,10 @@ jobs:
use-only-tar-bz2: true
if: ${{ env.IS_PYPY == 'false' }} # No pypy3.8 support

- name: Upgrade Arrow version
run: conda install -n pandas-dev -c conda-forge --no-update-deps pyarrow=${{ matrix.pyarrow_version }}
if: ${{ matrix.pyarrow_version }}

- name: Setup PyPy
uses: actions/setup-python@v2
with:
Expand Down
3 changes: 2 additions & 1 deletion pandas/tests/test_downstream.py
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,8 @@ def test_seaborn():


def test_pandas_gbq():

# Older versions import from non-public, non-existent pandas funcs
pytest.importorskip("pandas_gbq", minversion="0.10.0")
pandas_gbq = import_module("pandas_gbq") # noqa:F841


Expand Down

0 comments on commit ea0184e

Please sign in to comment.