Skip to content

Commit

Permalink
Gracefully add lint section into CI YAML
Browse files Browse the repository at this point in the history
Signed-off-by: Julio Faracco <[email protected]>
  • Loading branch information
jcfaracco committed Mar 10, 2024
1 parent 25bc9ea commit 347e805
Showing 1 changed file with 17 additions and 4 deletions.
21 changes: 17 additions & 4 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,13 @@ on:
pull_request: {}

jobs:
test_cpu:
runs-on: ubuntu-latest
lint:
environment: continuous_test
container:
image: docker.io/jcfaracco/dasf:cpu_ci
strategy:
matrix:
python-version: ['3.8', '3.9', '3.10']
os: [ubuntu-latest]
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v4
- name: Install dependencies
Expand All @@ -37,12 +39,23 @@ jobs:
run: |
# We should have at least 80% of docs covered
interrogate -vv -i --fail-under=15
test_cpu:
runs-on: ubuntu-latest
environment: continuous_test
needs: [lint]
container:
image: docker.io/jcfaracco/dasf:cpu_ci
steps:
- uses: actions/checkout@v4
- name: Run test cases
run: |
pytest tests/
test_gpu:
runs-on: ubuntu-latest
environment: continuous_test
needs: [lint]
# Skip this test for PRs due to dangerous code submitted.
if: ${{ github.event_name != 'pull_request' }}
steps:
Expand Down

0 comments on commit 347e805

Please sign in to comment.