Skip to content

Fix CI skip conditions in tests.yml#425

Merged
blink1073 merged 1 commit intoCalysto:mainfrom
blink1073:fix-skip-conditions
Mar 27, 2026
Merged

Fix CI skip conditions in tests.yml#425
blink1073 merged 1 commit intoCalysto:mainfrom
blink1073:fix-skip-conditions

Conversation

@blink1073
Copy link
Copy Markdown
Contributor

References

Description

Tightens skip conditions in the CI workflow so only steps that actually require GitHub secrets are conditionally skipped.

Changes

  • Removed overly broad if condition from build job (was skipping on scheduled runs outside the main repo)
  • Removed if: github.event_name == 'pull_request' from test_release job so it runs on all triggers
  • Removed allowed-skips: test_release from tests_check since test_release no longer skips
  • Added if condition to the Codecov upload step to skip on fork PRs (where CODECOV_TOKEN secret is unavailable)
  • Added missing persist-credentials: false to cover job's checkout step

Backwards-incompatible changes

None

Testing

N/A

AI usage

  • Some or all of the content of this PR was generated by AI.
  • The human author has carefully reviewed this PR and run this code.
  • AI tools and models used: Claude Sonnet 4.6 (Claude Code)

@blink1073 blink1073 enabled auto-merge (squash) March 27, 2026 11:43
@blink1073 blink1073 disabled auto-merge March 27, 2026 11:48
@blink1073 blink1073 merged commit d1a19c4 into Calysto:main Mar 27, 2026
22 checks passed
@blink1073 blink1073 deleted the fix-skip-conditions branch March 27, 2026 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant