Skip to content

Use early failure instead of job-level skip for fork-PR downstream tests#53

Merged
mtfishman merged 1 commit into
mainfrom
mf/early-failure
Mar 23, 2026
Merged

Use early failure instead of job-level skip for fork-PR downstream tests#53
mtfishman merged 1 commit into
mainfrom
mf/early-failure

Conversation

@mtfishman

Copy link
Copy Markdown
Member

Summary

  • Replace job-level skip with an early step that fails with a clear ::error:: message
  • The job now runs, so the conditional name expression resolves and the log is readable
  • The integration gate still blocks (failure instead of skip)

Test plan

🤖 Generated with Claude Code

…tream tests

Replace job-level skip with an early step that detects fork PR + URL
package, emits an ::error:: annotation with the /integrationtest
command, and exits with failure. This gives a readable log and error
message while still blocking the integration gate.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@mtfishman mtfishman merged commit 7bfe774 into main Mar 23, 2026
1 check passed
@mtfishman mtfishman deleted the mf/early-failure branch March 23, 2026 00:49
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.

1 participant