Skip to content

docs: point PR label guidance at REVIEWERS.md - #5403

Open
elezar wants to merge 1 commit into
mainfrom
docs/area-labels-source-of-truth/elezar
Open

elezar wants to merge 1 commit into
mainfrom
docs/area-labels-source-of-truth/elezar

Conversation

@elezar

@elezar elezar commented Nov 9, 2026

Copy link
Copy Markdown
Member

Summary

Tell contributors and agents to choose area: and topic: labels from the headings in REVIEWERS.md, which is what reviewer routing reads, instead of from the repository label list.

Related Issue

No linked issue; opened directly by a maintainer.

Changes

  • CONTRIBUTING.md: state that REVIEWERS.md is the source of truth for valid labels and that a label without a matching heading routes to no reviewers.
  • .agents/skills/create-github-pr/SKILL.md: add a Labels section with the same rule and name the legacy area:gateway label as an example to avoid.

Testing

  • Documentation-only change; the pre-commit hook ran on commit
  • Unit tests: not applicable
  • E2E tests: not applicable

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (not applicable)

Reviewer routing uses the area and topic headings in REVIEWERS.md, not the repository label list. State that in CONTRIBUTING.md and the create-github-pr skill.

Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar
elezar requested a review from purp November 9, 2026 15:42
@elezar

elezar commented Nov 9, 2026

Copy link
Copy Markdown
Member Author

@purp which area: would you assign to this PR?

@purp purp added the area:build Related to CI/CD and builds label Nov 9, 2026

@purp purp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 🚢

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:build Related to CI/CD and builds

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants