Skip to content

ci(labeler): update labels & associated files - #297

Merged
Ryan-Millard merged 2 commits into
mainfrom
ci/labeler/add-new-labels
Apr 7, 2026
Merged

ci(labeler): update labels & associated files#297
Ryan-Millard merged 2 commits into
mainfrom
ci/labeler/add-new-labels

Conversation

@Ryan-Millard

@Ryan-Millard Ryan-Millard commented Apr 6, 2026

Copy link
Copy Markdown
Owner

What was changed & why

Updates labeler workflow.
This provides better semantic labelling.

Fixes: none

Changes

Just the .github/labeler.yml and .github/workflows/pr-auto-label.yml

Testing & Verification

Additional Resources

Summary by CodeRabbit

  • Chores
    • Updated automated pull request labeling configuration to enhance repository organization and maintainability.

@github-actions github-actions Bot added the ci label Apr 6, 2026
@coderabbitai

coderabbitai Bot commented Apr 6, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

Updated GitHub Actions labeling configuration by restructuring label-matching rules in .github/labeler.yml, adding new component-based labels (webgpu, core, scripts, example-app, build-system), and expanding existing label patterns. Granted issues: write permission to the labeler workflow job.

Changes

Cohort / File(s) Summary
GitHub Actions Labeling Configuration
.github/labeler.yml, .github/workflows/pr-auto-label.yml
Restructured label-matching rules from nested to flattened format; expanded docs and js/ts pattern coverage; switched from package-lock.json/yarn.lock to pnpm-lock.yaml/pnpm-workspace.yaml; added five new labels (webgpu, core, scripts, example-app, build-system) with corresponding glob patterns; added issues: write permission to workflow job.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

ci

Poem

🐰 Labels hop and patterns dance,
New globs in YAML take their stance,
WebGPU shaders, core modules bright,
Scripts and apps now labeled right!
A workflow blessed with broader sight.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly references the main changes in the PR: updating CI labeler configuration and associated label patterns across two labeler-related files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/labeler/add-new-labels

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
.github/labeler.yml (1)

46-52: Tighten scripts globs to avoid broad labeling across the repository.

Lines 50–52 (*.sh, *.ps1, *.bat) match script files anywhere in the repository (e.g., scripts/deploy.sh, src/utils/helper.sh, root-level wrappers), not just the scripts/ directory. If the intent is to label only scripts directory changes, keep it scoped to scripts/** alone.

🔧 Proposed change
 scripts:
   - changed-files:
     - any-glob-to-any-file:
       - "scripts/**"
-      - "*.sh"
-      - "*.ps1"
-      - "*.bat"
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.github/labeler.yml around lines 46 - 52, The current labeler globs under
the "scripts" rule (the changed-files -> any-glob-to-any-file list) include
"*.sh", "*.ps1", and "*.bat" which match script files anywhere; restrict these
glob entries so they only target the scripts/ directory (or remove the
root-level globs) by updating the changed-files -> any-glob-to-any-file list to
reference only "scripts/**" (or equivalent scoped patterns) to avoid labeling
changes to script files outside the scripts/ folder.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In @.github/labeler.yml:
- Around line 46-52: The current labeler globs under the "scripts" rule (the
changed-files -> any-glob-to-any-file list) include "*.sh", "*.ps1", and "*.bat"
which match script files anywhere; restrict these glob entries so they only
target the scripts/ directory (or remove the root-level globs) by updating the
changed-files -> any-glob-to-any-file list to reference only "scripts/**" (or
equivalent scoped patterns) to avoid labeling changes to script files outside
the scripts/ folder.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 153b9ece-aa2c-48a1-a340-c1181e04490f

📥 Commits

Reviewing files that changed from the base of the PR and between d46a4c9 and 3ba332a.

📒 Files selected for processing (2)
  • .github/labeler.yml
  • .github/workflows/pr-auto-label.yml

@Ryan-Millard
Ryan-Millard merged commit 02b8bbf into main Apr 7, 2026
14 checks passed
@Ryan-Millard
Ryan-Millard deleted the ci/labeler/add-new-labels branch April 7, 2026 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant