Skip to content

ci(labeler): fix nested manifest globs and add api/shared/website area labels - #1454

Open
Wassim Chegham (manekinekko) wants to merge 2 commits into
mainfrom
manekinekko-pr-labels
Open

Wassim Chegham (manekinekko) wants to merge 2 commits into
mainfrom
manekinekko-pr-labels

Conversation

@manekinekko

Copy link
Copy Markdown
Member

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:

Labels on existing open PRs were updated separately; nothing in this PR touches them.

Demos

N/A. Repository configuration only.

Testing

Documentation and compatibility

None needed. CONTRIBUTING.md already defers to .github/labeler.yml for which labels the labeler manages. Once this merges, the labeler will add and remove the three new area: labels automatically on PR pushes, so they shouldn't be applied by hand.

Checklist

  • N/A: If Portal features changed, keep CLI capabilities in sync.
  • N/A: If Portal components changed, update their Storybook stories.
  • N/A: If database changes require a migration, include up() / down() and keep it CosmosDB-compatible.
  • N/A: If dependencies changed, update the lockfile and regenerate NOTICE / NOTICE-REVIEW.txt with pnpm notice as needed.
  • N/A: Video showing the behavior before the suggested change
  • N/A: Video showing the behavior after the suggested change

…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>

Copilot AI 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.

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

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: cicd Build, test, release, and deployment pipelines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants