Repository navigation
ci(labeler): fix nested manifest globs and add api/shared/website area labels - #1454
Open
Wassim Chegham (manekinekko) wants to merge 2 commits into
Open
Wassim Chegham (manekinekko) wants to merge 2 commits into
Wassim Chegham (manekinekko) wants to merge 2 commits into
Conversation
…I globs - Match Cargo.toml/Cargo.lock at any depth (gateway lives in apps/gateway), so sync-labels stops stripping type: dependencies from cargo bumps. - Treat pyproject.toml and uv.lock as dependency changes. - Limit topic: ui on the website to assets/components/scripts/styles instead of all of website/src (docs content, OpenAPI JSON). Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Wassim Chegham (manekinekko)
requested a review
from Cedric Vidal (cedricvidal)
as a code owner
October 1, 2026 19:35
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The labeler rules are valid, targeted, and consistent with existing configuration patterns.
Review effort: Balanced
Findings: None
What changed in this PR
Updates automated labeling to correctly classify nested dependency manifests and improve repository area coverage.
Changes:
- Supports nested Rust and Python dependency manifests.
- Narrows website UI matching to UI-specific directories.
- Adds API, shared-package, and website area labels.
| File | Description |
|---|---|
.github/labeler.yml |
Refines dependency/UI globs and adds three area mappings. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This fixes PR labeler rules that were dropping labels from Cargo and uv dependency bumps or adding wrong ones elsewhere, and adds three
area:labels for parts of the repo that had none.A scan of all 94 open PRs found these gaps:
type: dependencies/language: rust: these rules only matchedCargo.toml/Cargo.lockat the repo root, but the gateway's live inapps/gateway/. Becausesync-labels: trueremoves non-matching labels, cargo bumps (e.g. chore(deps): bump quinn-proto from 0.11.14 to 0.11.16 in /apps/gateway #1363) losttype: dependencieson every push. They now match**/Cargo.tomland**/Cargo.lock.type: dependencies: now also matches**/pyproject.tomland**/uv.lock(e.g. build(deps): bump pyjwt from 2.13.0 to 2.15.0 in /evaluations/static-prompts #1448, build(deps): bump urllib3 from 2.7.0 to 2.8.0 in /evaluations/static-prompts #1449 inevaluations/static-prompts).topic: ui: previously matched all ofwebsite/src/**, including docs content and the OpenAPI JSON. It now matches onlywebsite/src/{assets,components,scripts,styles}/**.area: api(apps/api/**),area: shared(packages/shared/**) andarea: website(website/**). The labels already exist in the repo. The scan found 22, 25 and 11 open PRs touching these paths, none of which had a fitting area label.Labels on existing open PRs were updated separately; nothing in this PR touches them.
Demos
N/A. Repository configuration only.
Testing
.github/labeler.ymlas YAML; all 21 label keys load.type: dependencieson chore(deps): bump quinn-proto from 0.11.14 to 0.11.16 in /apps/gateway #1363, build(deps): bump pyjwt from 2.13.0 to 2.15.0 in /evaluations/static-prompts #1448 and build(deps): bump urllib3 from 2.7.0 to 2.8.0 in /evaluations/static-prompts #1449, and notopic: uion docs-only website PRs such as docs: document model lifecycle and deprecation #1435.Documentation and compatibility
None needed.
CONTRIBUTING.mdalready defers to.github/labeler.ymlfor which labels the labeler manages. Once this merges, the labeler will add and remove the three newarea:labels automatically on PR pushes, so they shouldn't be applied by hand.Checklist
up()/down()and keep it CosmosDB-compatible.NOTICE/NOTICE-REVIEW.txtwithpnpm noticeas needed.