From d51de17602fedc4c5797c97845dc7cd99da4b9c4 Mon Sep 17 00:00:00 2001 From: Erich <137924413+cb-ekuersch@users.noreply.github.com> Date: Thu, 28 May 2026 15:54:57 -0500 Subject: [PATCH] ci: narrow illustrations-icons-checklist triggers to src/ paths only (#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> --- .github/workflows/illustrations-icons-checklist.yml | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/.github/workflows/illustrations-icons-checklist.yml b/.github/workflows/illustrations-icons-checklist.yml index 1e10a96bca..4ff3230950 100644 --- a/.github/workflows/illustrations-icons-checklist.yml +++ b/.github/workflows/illustrations-icons-checklist.yml @@ -3,6 +3,9 @@ name: Enforce illustrations/icons checklist on: pull_request: types: [opened, edited, synchronize, reopened] + paths: + - 'packages/illustrations/src/**' + - 'packages/icons/src/**' permissions: contents: read @@ -29,9 +32,9 @@ jobs: { owner, repo, pull_number, per_page: 100 } ); - const relevant = files.some(f => /^packages\/(illustrations|icons)\//.test(f.filename)); + const relevant = files.some(f => /^packages\/(illustrations|icons)\/src\//.test(f.filename)); if (!relevant) { - core.info('No changes under packages/illustrations or packages/icons. Skipping.'); + core.info('No changes under packages/illustrations/src or packages/icons/src. Skipping.'); return; }