Skip to content

ci(infra): drop redundant opened trigger from issue-link check#36054

Merged
Mason Daugherty (mdrxy) merged 1 commit intomasterfrom
mdrxy/fix-skip
Mar 18, 2026
Merged

ci(infra): drop redundant opened trigger from issue-link check#36054
Mason Daugherty (mdrxy) merged 1 commit intomasterfrom
mdrxy/fix-skip

Conversation

@mdrxy
Copy link
Member

Remove opened from the require_issue_link.yml trigger list. New PRs never have labels at creation time, so the job-level if (which requires the external label) always evaluated to skip — producing a noisy "Skipped" check on every internal PR. Enforcement chains through pr_labeler.ymllabeled event, making opened a no-op.

Changes

  • Drop opened from pull_request_target.types in require_issue_link.yml — only edited, reopened, and labeled remain
  • Update header comment to document why opened is intentionally excluded

@github-actions github-actions bot added github_actions Pull requests that update GitHub Actions code infra PRs made that include chores, devops, repo meta changes internal size: XS < 50 LOC labels Mar 18, 2026
@mdrxy Mason Daugherty (mdrxy) merged commit 54a5f83 into master Mar 18, 2026
93 checks passed
@mdrxy Mason Daugherty (mdrxy) deleted the mdrxy/fix-skip branch March 18, 2026 02:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code infra PRs made that include chores, devops, repo meta changes internal size: XS < 50 LOC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant