Skip to content

Commit

Permalink
Rename to setup
Browse files Browse the repository at this point in the history
  • Loading branch information
hoxbro committed Dec 21, 2023
1 parent 9a662f1 commit 60b3c70
Showing 1 changed file with 11 additions and 10 deletions.
21 changes: 11 additions & 10 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,17 +21,18 @@ jobs:
runs-on: 'ubuntu-latest'
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 }}
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 @@ -44,9 +45,9 @@ jobs:
- '.github/workflows/test.yaml'
unit_test_suite:
name: Unit tests on ${{ matrix.os }} with Python ${{ matrix.python-version }}
needs: [pre_commit, changes]
needs: [pre_commit, setup]
runs-on: ${{ matrix.os }}
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
strategy:
fail-fast: false
matrix:
Expand Down Expand Up @@ -120,9 +121,9 @@ jobs:
fail_ci_if_error: false # optional (default = false)
ui_test_suite:
name: UI tests on ${{ matrix.os }} with Python 3.9
needs: [pre_commit, changes]
needs: [pre_commit, setup]
runs-on: ${{ matrix.os }}
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
strategy:
fail-fast: false
matrix:
Expand Down Expand Up @@ -222,9 +223,9 @@ jobs:
fail_ci_if_error: false # optional (default = false)
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 }}
if: needs.changes.outputs.code == 'true'
if: needs.setup.outputs.code_change == 'true'
strategy:
fail-fast: false
matrix:
Expand Down

0 comments on commit 60b3c70

Please sign in to comment.