Repository navigation
CONTRIBUTING: use the semver- label names from .autorc - #455
Conversation
The documented labels (major, minor, patch, ...) do not exist in the repo; .autorc maps release types to semver-major, semver-minor, etc. Drop skip-release, which is neither a repo label nor in .autorc. Co-Authored-By: Claude Code 2.1.285 / Claude Opus 5.5 <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #455 +/- ##
=======================================
Coverage 92.03% 92.03%
=======================================
Files 15 15
Lines 1344 1344
Branches 182 182
=======================================
Hits 1237 1237
Misses 71 71
Partials 36 36 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
It is a documentation-only change whose every renamed label was verified to match the names defined in .autorc.
Review effort: Balanced
Findings: None
What changed in this PR
This PR corrects the PR-label documentation in CONTRIBUTING.rst. The repository uses auto (configured via .autorc) to classify changelog entries and drive releases, and .autorc defines the semver labels with a semver- prefix (e.g. semver-major, semver-patch). The prior documentation listed unprefixed names (major, minor, patch, …) that don't exist in the repo, so following the docs (e.g. gh pr edit --add-label minor) failed. The change aligns the docs with the actual label names and removes a stale entry.
Changes:
- Renamed the documented labels to their
semver-equivalents and updated the "unlabelled means patch" sentence tosemver-patch, with a pointer to.autorc. - Removed the
skip-releaseentry, which is neither a repo label nor defined in.autorc(and is redundant givenonlyPublishWithReleaseLabel). - Kept the special
releaselabel unchanged.
| File | Description |
|---|---|
| CONTRIBUTING.rst | Updated the recognized PR-label list and default-label sentence to match the semver--prefixed names defined in .autorc, and dropped the nonexistent skip-release label. |
Verification notes: I confirmed against .autorc that all eight documented labels (semver-major, semver-minor, semver-patch, semver-internal, semver-documentation, semver-tests, semver-dependencies, semver-performance) match the configured name values exactly. The release label is auto's built-in publish trigger, so keeping it is correct, and skip-release is correctly removed.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
CONTRIBUTING.rst lists PR labels as
major,minor,patch, ..., but those labels do not exist in the repo:.autorcmaps release types tosemver-major,semver-minor, etc. Following the doc gives'minor' not foundfromgh pr edit --add-label.semver-names (and the "unlabelled means patch" sentence), with a pointer to.autorc.skip-release: it is neither a repo label nor in.autorc, and withonlyPublishWithReleaseLabelnothing releases withoutreleaseanyway.Verified against
.autorcandgh label list -R con/duct.🤖 Generated with Claude Code