Skip to content

Commit

Permalink
ci: Temporarily deactivate coverage comments on PRs from fork (#1042)
Browse files Browse the repository at this point in the history
Temporarily deactivate coverage comments on PRs from fork, which don't
have enough permissions.
  • Loading branch information
thomass-dev authored Jan 3, 2025
1 parent ef116ed commit f1e0fe5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/skore-ui.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ jobs:
npm install
npm run test:unit:coverage
- name: Report coverage
if: always()
if: ${{ always() && ! github.event.pull_request.head.repo.fork }}
uses: davelosert/vitest-coverage-report-action@v2
with:
working-directory: ./skore-ui
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/skore.yml
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@ jobs:
# run coverage
python -m pytest -n auto --junitxml=coverage.xml --cov=skore src/ tests/ | tee pytest-coverage.txt
- name: Pytest coverage comment
if: ${{ ! github.event.pull_request.head.repo.fork }}
uses: MishaKav/pytest-coverage-comment@main
with:
pytest-coverage-path: ./skore/pytest-coverage.txt
Expand Down

0 comments on commit f1e0fe5

Please sign in to comment.