Skip to content

Commit

Permalink
Update namings
Browse files Browse the repository at this point in the history
  • Loading branch information
hoxbro committed Dec 21, 2023
1 parent 79cc55b commit 3c45b01
Showing 1 changed file with 30 additions and 29 deletions.
59 changes: 30 additions & 29 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -46,18 +46,19 @@ jobs:
steps:
- uses: holoviz-dev/holoviz_tasks/[email protected]

changes:
name: Check for code changes
setup:
name: Setup workflow
runs-on: ubuntu-latest
permissions:
pull-requests: read
outputs:
code: ${{ steps.filter.outputs.code }}
code_change: ${{ steps.filter.outputs.code }}
matrix: ${{ env.MATRIX }}
steps:
- uses: actions/checkout@v3
if: github.event_name != 'pull_request'
- uses: dorny/[email protected]
- name: Check for code changes
uses: dorny/[email protected]
id: filter
with:
filters: |
Expand All @@ -67,29 +68,29 @@ jobs:
- 'setup.py'
- 'pyproject.toml'
- '.github/workflows/test.yaml'
- name: set matrix - default
- name: Set test matrix with 'default' option
if: github.event_name != 'workflow_dispatch' || (github.event_name == 'workflow_dispatch' && github.event.inputs.target == 'default')
run: |
MATRIX=$(jq -nsc '{
"os": ["ubuntu-latest", "macos-latest", "windows-latest"],
"python-version": ["3.9", "3.11"],
"include": [
{
"os": "windows-latest",
"python-version": "3.9"
"os": "ubuntu-latest",
"python-version": "3.10"
}
]
}')
echo "MATRIX=$MATRIX" >> $GITHUB_ENV
- name: set matrix - full
- name: Set test matrix with 'full' option
if: github.event_name == 'workflow_dispatch' && github.event.inputs.target == 'full'
run: |
MATRIX=$(jq -nsc '{
"os": ["ubuntu-latest", "macos-latest", "windows-latest"],
"python-version": ["3.9", "3.10", "3.11"]
}')
echo "MATRIX=$MATRIX" >> $GITHUB_ENV
- name: set matrix - downstream
- name: Set test matrix with 'downstream' option
if: github.event_name == 'workflow_dispatch' && github.event.inputs.target == 'downstream'
run: |
MATRIX=$(jq -nsc '{
Expand All @@ -100,18 +101,18 @@ jobs:
unit_test_suite:
name: Unit tests on Python ${{ matrix.python-version }}, ${{ matrix.os }}
needs: [pre_commit, changes]
needs: [pre_commit, setup]
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix: ${{ fromJson(needs.changes.outputs.matrix) }}
matrix: ${{ fromJson(needs.setup.outputs.matrix) }}
timeout-minutes: 120
env:
DESC: "Python ${{ matrix.python-version }}, ${{ matrix.os }} unit tests"
PYTHON_VERSION: ${{ matrix.python-version }}
steps:
- uses: holoviz-dev/holoviz_tasks/[email protected]
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
with:
name: unit_test_suite
python-version: ${{ matrix.python-version }}
Expand All @@ -122,29 +123,29 @@ jobs:
conda-update: true
id: install
- name: bokeh sampledata
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
run: |
conda activate test-environment
bokeh sampledata
- name: doit test_unit
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
run: |
conda activate test-environment
doit test_unit
- name: test examples
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
run: |
conda activate test-environment
doit test_examples
- name: codecov
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
run: |
conda activate test-environment
codecov
ui_test_suite:
name: UI tests on Python ${{ matrix.python-version }}, ${{ matrix.os }}
needs: [pre_commit, changes]
needs: [pre_commit, setup]
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
Expand All @@ -161,7 +162,7 @@ jobs:
PYCTDEV_SELF_CHANNEL: "pyviz/label/dev"
steps:
- uses: holoviz-dev/holoviz_tasks/[email protected]
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
with:
name: ui_test_suite
python-version: ${{ matrix.python-version }}
Expand All @@ -171,12 +172,12 @@ jobs:
playwright: true
id: install
- name: doit test_ui
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
run: |
conda activate test-environment
doit test_ui
- name: Upload coverage to Codecov
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
uses: codecov/codecov-action@v3
with:
files: ./coverage.xml
Expand All @@ -185,7 +186,7 @@ jobs:

core_test_suite:
name: Core tests on Python ${{ matrix.python-version }}, ${{ matrix.os }}
needs: [pre_commit, changes]
needs: [pre_commit, setup]
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
Expand All @@ -199,7 +200,7 @@ jobs:
steps:
# Add back when this works on Python 3.12
# - uses: holoviz-dev/holoviz_tasks/[email protected]
# if: needs.changes.outputs.code == 'true'
# if: needs.setup.outputs.code_change == 'true'
# with:
# name: core_test_suite
# python-version: ${{ matrix.python-version }}
Expand All @@ -210,32 +211,32 @@ jobs:
# conda-update: true
# id: install
- uses: actions/checkout@v3
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
with:
fetch-depth: "100"
- name: Fetch unshallow
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
run: git fetch --prune --tags --unshallow -f
- uses: actions/setup-python@v4
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
with:
python-version: 3.12
- name: install
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
run: |
python -m pip install -ve '.[tests_core, tests_ci]'
- name: bokeh sampledata
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
run: |
# conda activate test-environment
bokeh sampledata
- name: Check packages latest version
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
run: |
# conda activate test-environment
python scripts/check_latest_packages.py
- name: doit test_unit
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
run: |
# conda activate test-environment
pytest holoviews

0 comments on commit 3c45b01

Please sign in to comment.