Skip to content

Remove org-wide labeling; it belongs to coreruleset only - #2

Open
fzipi wants to merge 1 commit into
mainfrom
scope-labeling-to-coreruleset
Open

Remove org-wide labeling; it belongs to coreruleset only#2
fzipi wants to merge 1 commit into
mainfrom
scope-labeling-to-coreruleset

Conversation

@fzipi

@fzipi fzipi commented Sep 8, 2026

Copy link
Copy Markdown
Member

Summary

  • go-ftw, crs-toolchain, and plugin-registry already have labels literally named release:fix, release:ignore, release:new-feature, and :book: documentation for unrelated purposes, so the org-wide labeling_instructions was mislabeling PRs in those repos, not safely no-op-ing
  • The schema has no per-repository scoping field, so suggested_labels, auto_apply_labels, labeling_instructions, and mutually_exclusive_groups are removed from this org config
  • Labeling moves to a repo-local .coderabbit.yaml in coreruleset/coreruleset (companion PR: coreruleset/coreruleset#TBD), which sets inheritance: true to keep everything else from this org config
  • slop_detection (the ai-slop label) stays org-wide — no collision found across the org's repos

Test plan

  • yamllint -d relaxed .coderabbit.yaml — only pre-existing line-length warnings
  • check-jsonschema against the CodeRabbit schema — passes
  • gh label list across go-ftw, crs-toolchain, crs-linter, albedo, rassemble-go, plugin-registry, msc_pyparser, coraza-coreruleset confirmed the label-name collisions cited above

Confirmed go-ftw, crs-toolchain, and plugin-registry already have
labels with the exact same names our release/topic taxonomy uses
(release:fix, release:ignore, release:new-feature, 📖
documentation), for unrelated purposes. The "missing label no-ops"
assumption only protects against typos, not real collisions, so the
org-wide config was mislabeling PRs in those repos. The schema has no
per-repository scoping field, so labeling moves to a repo-local
.coderabbit.yaml in coreruleset/coreruleset instead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant