Skip to content

ci: don't run coverage workflow on forks#263

Merged
jolars merged 1 commit intofrederikfabriciusbjerre:mainfrom
jolars:fix-codecov-workflow
Mar 30, 2026
Merged

ci: don't run coverage workflow on forks#263
jolars merged 1 commit intofrederikfabriciusbjerre:mainfrom
jolars:fix-codecov-workflow

Conversation

@jolars
Copy link
Copy Markdown
Collaborator

@jolars jolars commented Mar 30, 2026

This is a small fix to the workflow to stop it running on forks, which wastes github action runner budgets and always fails, sending an annoying email (at least for me).

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Codecov/coverage GitHub Actions workflow to avoid running coverage jobs on forked repositories, reducing wasted CI resources and noisy failures for fork owners.

Changes:

  • Added job-level if: conditions intended to skip coverage jobs when the repository is a fork.
  • Applied the same gating to both the R coverage job and the Rust coverage job.

Comment thread .github/workflows/test-coverage.yaml
Comment thread .github/workflows/test-coverage.yaml
@jolars jolars requested a review from BjarkeHautop March 30, 2026 08:15
@jolars jolars merged commit 6216937 into frederikfabriciusbjerre:main Mar 30, 2026
13 checks passed
@jolars jolars deleted the fix-codecov-workflow branch March 30, 2026 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants