Skip to content

docs: add versioning and release-impact convention to CLAUDE.md - #99

Merged
PiVortex merged 1 commit into
mainfrom
docs/versioning-convention
Jun 17, 2026
Merged

PiVortex merged 1 commit into
mainfrom
docs/versioning-convention

Conversation

@PiVortex

Copy link
Copy Markdown
Collaborator

What & why

We bump published-package versions in dedicated chore(release): commits on main, not in feature PRs — but that wasn't written down, so reviewers (and agents) flag "no version bump" on PRs like #95. This records the convention and adds a PR-description requirement so release impact is signalled up front.

Change

Adds a "Versioning and releases" section to the root CLAUDE.md:

  • Never bump a package version in a PR into main — versions for @neardefi/shade-agent-js, @neardefi/shade-agent-cli, and the shade-attestation crate are bumped separately on main via chore(release): <package>-vX.Y.Z commits.
  • PRs that change a published package must state the expected release impact (which package + major/minor/patch, per semver) in the description, unless it already says so. PRs touching no published package can say "no release impact".

Release impact

No release impact — docs/convention only, no published package touched.

🤖 Generated with Claude Code

Codify that package versions are never bumped in PRs into main (handled
separately on main via chore(release) commits), and that PRs touching a
published package must state the expected major/minor/patch release impact.
@PiVortex
PiVortex merged commit 7c4597f into main Jun 17, 2026
7 checks passed
@PiVortex
PiVortex deleted the docs/versioning-convention branch June 17, 2026 10:42
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