Skip to content

Add descriptive job name for skipped fork-PR downstream tests#52

Merged
mtfishman merged 1 commit into
mainfrom
mf/remove-notify-skip
Mar 22, 2026
Merged

Add descriptive job name for skipped fork-PR downstream tests#52
mtfishman merged 1 commit into
mainfrom
mf/remove-notify-skip

Conversation

@mtfishman

Copy link
Copy Markdown
Member

Summary

  • When a URL-based package is skipped on a fork PR, show "(fork PR - use /integrationtest to run)" in the job name
  • For normal runs, the job name remains the package name
  • Testing whether conditional name expressions resolve for skipped jobs

Test plan

🤖 Generated with Claude Code

When a URL-based package is skipped on a fork PR, show
"(fork PR - use /integrationtest to run)" in the job name.
Testing whether name expressions resolve for skipped jobs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@mtfishman mtfishman force-pushed the mf/remove-notify-skip branch from 0435eb5 to 6b3be3f Compare March 22, 2026 23:54
@mtfishman mtfishman merged commit 4550e25 into main Mar 22, 2026
1 check passed
@mtfishman mtfishman deleted the mf/remove-notify-skip branch March 22, 2026 23:54
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