Skip to content

CONTRIBUTING: use the semver- label names from .autorc - #455

Merged
asmacdo merged 1 commit into
con:mainfrom
asmacdo:contributing-semver-labels
Sep 30, 2026
Merged

asmacdo merged 1 commit into
con:mainfrom
asmacdo:contributing-semver-labels

Conversation

@asmacdo

@asmacdo asmacdo commented Sep 30, 2026

Copy link
Copy Markdown
Member

CONTRIBUTING.rst lists PR labels as major, minor, patch, ..., but those labels do not exist in the repo: .autorc maps release types to semver-major, semver-minor, etc. Following the doc gives 'minor' not found from gh pr edit --add-label.

  • Rename the documented labels to their semver- names (and the "unlabelled means patch" sentence), with a pointer to .autorc.
  • Drop skip-release: it is neither a repo label nor in .autorc, and with onlyPublishWithReleaseLabel nothing releases without release anyway.

Verified against .autorc and gh label list -R con/duct.

🤖 Generated with Claude Code

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>
@asmacdo asmacdo added the semver-documentation Changes only affect the documentation label Sep 30, 2026
@codecov

codecov Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.03%. Comparing base (7a25f42) to head (80ec9ca).
⚠️ Report is 3 commits behind head on main.

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.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@asmacdo
asmacdo marked this pull request as ready for review September 30, 2026 19:02
Copilot AI balanced review requested due to automatic review settings September 30, 2026 19:02

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

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 to semver-patch, with a pointer to .autorc.
  • Removed the skip-release entry, which is neither a repo label nor defined in .autorc (and is redundant given onlyPublishWithReleaseLabel).
  • Kept the special release label 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.

@asmacdo
asmacdo merged commit 7ad5f79 into con:main Sep 30, 2026
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver-documentation Changes only affect the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants