Skip to content

skip coverage on forks#1387

Merged
pearce8 merged 1 commit into
developfrom
fork-coverage
Jun 12, 2026
Merged

skip coverage on forks#1387
pearce8 merged 1 commit into
developfrom
fork-coverage

Conversation

@michaelmckinsey1

@michaelmckinsey1 michaelmckinsey1 commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Description

  • Currently, dryruns always appear as failure on forks because the coverage step will always fail. This skips coverage on forks, so we can still easily tell if the unit tests are passing.

I made a demo PR #1388 to show this works (this is the same PR but from a fork) and the tests are skipped there.

@michaelmckinsey1 michaelmckinsey1 self-assigned this Jun 12, 2026
@github-actions github-actions Bot added the ci CI, unit tests, GitHub actions label Jun 12, 2026
@michaelmckinsey1

Copy link
Copy Markdown
Collaborator Author

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.36%. Comparing base (2b08198) to head (65d1193).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1387      +/-   ##
===========================================
+ Coverage    62.82%   64.36%   +1.54%     
===========================================
  Files           49       49              
  Lines         3822     3822              
  Branches       339      339              
===========================================
+ Hits          2401     2460      +59     
+ Misses        1402     1343      -59     
  Partials        19       19              

see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pearce8 pearce8 added this pull request to the merge queue Jun 12, 2026
Merged via the queue into develop with commit 8e9b897 Jun 12, 2026
22 checks passed
@pearce8 pearce8 deleted the fork-coverage branch June 12, 2026 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci CI, unit tests, GitHub actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants