Skip to content

Commit

Permalink
Let the main CI workflow invoke the lint workflow (#359)
Browse files Browse the repository at this point in the history
  • Loading branch information
achimnol authored Apr 26, 2023
1 parent f5200c2 commit 1e6419d
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
5 changes: 4 additions & 1 deletion .github/workflows/ci-cd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,9 @@ jobs:
path: dist/${{ env.sdist-name }}
retention-days: 15

lint:
uses: ./.github/workflows/lint.yml

test-pytest:
name: Tests on ${{ matrix.python-version }}
needs: build-sdist
Expand Down Expand Up @@ -92,7 +95,7 @@ jobs:
check: # This job does nothing and is only used for the branch protection
name: ✅ Ensure the required checks passing
if: always()
needs: [build-sdist, test-pytest]
needs: [build-sdist, lint, test-pytest]
runs-on: ubuntu-latest
steps:
- name: Decide whether the needed jobs succeeded or failed
Expand Down
5 changes: 1 addition & 4 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,7 @@
name: Linters

on:
push:
branches: ["main"]
pull_request:
branches: ["main"]
workflow_call:
workflow_dispatch:

env:
Expand Down

0 comments on commit 1e6419d

Please sign in to comment.