Skip to content

Commit

Permalink
ci: make test dependent on build step
Browse files Browse the repository at this point in the history
  • Loading branch information
ze-flo committed Jun 11, 2024
1 parent e8c3827 commit d88f5df
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,7 @@ jobs:
run: npm run lint:ci

test:
needs: [build]
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
Expand All @@ -80,6 +81,12 @@ jobs:
- name: Install Dependencies
run: npm ci --ignore-scripts

- name: Download dist artifacts
uses: actions/download-artifact@v4
with:
name: dist-artifact
path: packages

- name: Test
run: npm run test:ci

Expand All @@ -90,7 +97,7 @@ jobs:
COVERALLS_REPO_TOKEN: ${{ secrets.COVERALLS_REPO_TOKEN }}

deploy:
needs: [test, build-demo]
needs: [build-demo, lint, test]
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
Expand Down Expand Up @@ -118,7 +125,7 @@ jobs:
NETLIFY_SITE_ID: ${{ secrets.NETLIFY_SITE_ID }}

publish:
needs: [test, build]
needs: [build, lint, test]
runs-on: ubuntu-latest
# if: github.ref == 'refs/heads/main'
steps:
Expand Down

0 comments on commit d88f5df

Please sign in to comment.