Skip to content

ci: narrow illustrations-icons-checklist triggers to src/ paths only (#735) (backport to release-8.x)#737

Merged
cb-ekuersch merged 1 commit into
release-8.xfrom
backport/e58976ea-to-release-8.x
May 28, 2026
Merged

ci: narrow illustrations-icons-checklist triggers to src/ paths only (#735) (backport to release-8.x)#737
cb-ekuersch merged 1 commit into
release-8.xfrom
backport/e58976ea-to-release-8.x

Conversation

@cb-ekuersch
Copy link
Copy Markdown
Contributor

Summary

Backport of commit e58976ea from master, originally merged via #735.

  • Adds a paths filter to the illustrations-icons-checklist workflow's pull_request trigger so it only runs when files under packages/illustrations/src/** or packages/icons/src/** change.
  • Tightens the in-script relevance regex from packages/(illustrations|icons)/ to packages/(illustrations|icons)/src/ so the trigger and script gates stay consistent.

Test plan

  • Verify no CI steps in the release-8.x pipeline depend on anything removed or changed by this commit

🤖 Generated with Claude Code

…735)

The workflow was firing on any file change inside packages/illustrations/
or packages/icons/, including package.json, tsconfig, READMEs, Babel
config, and other metadata. This caused unnecessary CI noise on routine
config edits.

Two changes were made:
- Added a `paths` filter to the pull_request trigger so it only
  activates when files under packages/illustrations/src/** or
  packages/icons/src/** change.
- Updated the in-script relevance regex from
  `packages/(illustrations|icons)/` to
  `packages/(illustrations|icons)/src/` so both gates are consistent.

Co-authored-by: forge[bot] <1548036+forge[bot]@users.noreply.ghe.com>
@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented May 28, 2026

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 1
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1
CODEOWNERS ✅ See below

⚠️ Ignored Reviews (2)

Reviewer Reason
hcopp Bound to another issue (#735)
hcopp Review missing MFA

CODEOWNERS

Code Owner Status Calculation
ui-systems-eng-team 1/1
Denominator calculation
Additional CODEOWNERS Requirement
Show calculation
Sum 0
0
From CODEOWNERS 1
Sum 1

@cb-ekuersch cb-ekuersch merged commit b836414 into release-8.x May 28, 2026
16 checks passed
@cb-ekuersch cb-ekuersch deleted the backport/e58976ea-to-release-8.x branch May 28, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants