Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CI test to use holoviz_tasks/install #607

Merged
merged 6 commits into from
Jan 3, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
47 changes: 11 additions & 36 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,43 +43,22 @@ jobs:
env:
DISPLAY: ":99.0"
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
MAMBA_NO_BANNER: 1
SETUPTOOLS_ENABLE_FEATURES: "legacy-editable"
steps:
- uses: actions/checkout@v3
with:
fetch-depth: "100"
- uses: actions/setup-python@v4
- uses: pyviz-dev/holoviz_tasks/install@v0.1a9
with:
name: unit_test_suite
python-version: ${{ matrix.python-version }}
- uses: conda-incubator/setup-miniconda@v2
with:
miniconda-version: "latest"
- name: Fetch unshallow
run: git fetch --prune --tags --unshallow
- name: conda setup
run: |
conda install -n base -c conda-forge mamba --no-update-deps
conda install -c conda-forge "nodejs=15.3.0" --no-update-deps
conda create -n test-environment
conda activate test-environment
conda config --env --append channels pyviz/label/dev --append channels conda-forge
conda config --env --remove channels defaults
conda install python=${{ matrix.python-version }} pyctdev
- name: doit develop_install
if: matrix.os != 'macos-latest'
run: |
conda activate test-environment
doit develop_install -o tests -o examples -o recommended --conda-mode=mamba
# Temporary hacked step as on MacOS doit develop_install updated CPython leading to a pyctdev failure
- name: doit develop_install
if: matrix.os == 'macos-latest'
run: |
conda activate test-environment
doit develop_install -o tests -o examples -o recommended --conda-mode=mamba || echo "Keep going"
pip install --no-deps --no-build-isolation -e .
channel-priority: strict
channels: pyviz/label/dev,conda-forge,nodefaults
envs: "-o tests -o examples -o recommended"
cache: true
conda-update: true
conda-mamba: mamba
nodejs: true
id: install
- name: patch fiona/geostack on Python 3.7 / Macos
if: contains(matrix.os, 'macos') && matrix.python-version == '3.7'
if: steps.install.outputs.cache-hit != 'true' && contains(matrix.os, 'macos') && matrix.python-version == '3.7'
run: |
conda activate test-environment
mamba install "fiona=1.8" "gdal=3.3"
Expand All @@ -95,10 +74,6 @@ jobs:
- name: git describe
run: |
git describe
- name: doit test_flakes
run: |
conda activate test-environment
doit test_flakes
- name: doit test_unit
run: |
conda activate test-environment
Expand Down
4 changes: 2 additions & 2 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@ repos:
rev: 4.0.1
hooks:
- id: flake8 # See 'setup.cfg' for args
args: [holoviews]
files: holoviews/
args: [geoviews]
files: geoviews/
- repo: https://github.com/hoxbro/clean_notebook
rev: 0.1.5
hooks:
Expand Down