Skip to content

Commit

Permalink
fixup! TST: simplify cartopy integration testing (using cartopy 0.22)
Browse files Browse the repository at this point in the history
  • Loading branch information
neutrinoceros committed Aug 5, 2023
1 parent eb3c24c commit f19fc57
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ jobs:
- name: Install dependencies and yt
shell: bash
env:
dependencies: ''
dependencies: 'cartopy'
run: |
source ./tests/ci_install.sh
python -m pip install 'cartopy>=0.22'
Expand Down
5 changes: 4 additions & 1 deletion tests/ci_install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ set -x # Show which command is being run
cp tests/matplotlibrc .

# Step 1: pre-install required packages
if [[ ${dependencies} == "full" ]]; then
if [[ ${dependencies} == "full" || ${dependencies} == "cartopy" ]]; then
# upgrading pip to guarantee installing extra dependencies with [full] is supported
# this is only necessary for some versions of Python 3.8 and 3.9
# see https://github.com/yt-project/yt/issues/4270
Expand Down Expand Up @@ -41,6 +41,9 @@ fi
if [[ ${dependencies} == "minimal" ]]; then
# test with minimal versions of runtime dependencies
python -m pip install -e ".[test,minimal]"
elif [[ ${dependencies} == "cartopy" ]]; then
python -m pip install 'cartopy>=0.22'
python -m pip install -e ".[test]"
elif [[ ${dependencies} == "full" ]]; then
# test with all optional runtime dependencies
python -m pip install 'cartopy>=0.22'
Expand Down

0 comments on commit f19fc57

Please sign in to comment.