Skip to content

Document the semantic, maintainer-controlled versioning policy - #131

Merged
ptr727 merged 6 commits into
developfrom
document-versioning-policy
Jun 8, 2026
Merged

Document the semantic, maintainer-controlled versioning policy#131
ptr727 merged 6 commits into
developfrom
document-versioning-policy

Conversation

@ptr727

@ptr727 ptr727 commented Jun 8, 2026

Copy link
Copy Markdown
Owner

Documents how version.json is managed, so the versioning model is explicit and propagates to derived repos.

  • develop leads main by a minor. main builds the stable release (X.Y.height); develop builds prereleases (X.Y.height-g{sha}). After each develop -> main release, bump develop's minor in an isolated bump-version-X.Y PR so develop's prereleases always sort numerically above main's last stable (at a shared version, develop's prerelease sorts below main's stable - confusing in --version, HISTORY.md, update prompts).
  • Maintenance promotions hold main's version. Catch-up work (dependency bumps, CI/doc fixes, template re-syncs) reaches main without releasing develop's leading minor via git checkout main -- version.json on the promotion branch (the pattern used for PlexCleaner Promote the Fleet Resync Reports, the copilot-instructions Fixes, and the Upstream Repo-Layout Segment #734). A release promotion carries develop's version, after which develop bumps again.

Written in current-state declarative voice. No CI enforcement (documentation only).

Make explicit how version.json is managed: the major.minor floor is bumped
deliberately by the maintainer for real releases (features / significant
changes), while routine maintenance (dependency bumps, CI/doc fixes, template
re-syncs) lands on main as a patch and never bumps the minor. develop and main
normally share the same version (develop is the prerelease channel); develop is
not required to lead main and should not be preemptively bumped. Document the
one-off 'hold version.json' step for promoting maintenance when develop's
version was already bumped ahead - framed as the exception, not the routine.
Copilot AI review requested due to automatic review settings June 8, 2026 17:17

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.

Pull request overview

Documents the repository's semantic, maintainer-controlled versioning policy for version.json/NBGV, aiming to prevent accidental minor bumps during routine maintenance.

Changes:

  • Add guidance that version.json (major.minor) is a deliberate floor and should only be bumped for intentional feature releases.
  • Clarify that routine maintenance should not change version.json and should ship as patches of the current major.minor.
  • Document the expected relationship between develop and main versioning and how to handle the exceptional case where develop is bumped ahead.

Comment thread AGENTS.md Outdated
Describe the build version conceptually (major.minor.height with a prerelease
suffix off main, NBGV-computed) instead of asserting an exact prerelease string
format that doesn't match what NBGV actually produces.
Drop the change-from-prior framing (no 'not required to lead', 'don't
preemptively bump', 'exception not the routine') so the doc states only what is,
consistent with the Documentation Style Conventions.

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread AGENTS.md Outdated
main builds stable, develop builds prereleases. develop leads main by a minor:
after each develop -> main release, bump develop's minor in an isolated PR so
develop prereleases sort above main's last stable (a shared version sinks
develop's prerelease below main's stable in --version/HISTORY/update prompts).
Maintenance promotions hold main's version (git checkout main -- version.json)
so catch-up work lands as a patch without releasing develop's leading minor.
Copilot AI review requested due to automatic review settings June 8, 2026 17:50

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
State the develop-leads rule in all three cross-cutting docs: a develop -> main
release is followed by a develop minor bump (isolated bump-version PR) so develop
prereleases sort above main's stable, and maintenance promotions hold main's
version.

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.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comment thread AGENTS.md Outdated
Use <...> placeholders (X.Y.<height>-g<sha>) consistent with the rest of the
doc, and state that a held-version maintenance promotion advances main only by
its NBGV git height (the SemVer patch position), not a manual patch bump.

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.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

@ptr727
ptr727 requested a review from Copilot June 8, 2026 17:58
@ptr727
ptr727 merged commit 794850f into develop Jun 8, 2026
9 checks passed

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.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

ptr727 added a commit that referenced this pull request Jun 8, 2026
Promotes the versioning-policy documentation to `main` (#131): the
develop-leads + maintenance-hold model stated in AGENTS.md, README.md,
and .github/copilot-instructions.md. Documentation only; version
unchanged (maintenance).
@ptr727
ptr727 deleted the document-versioning-policy branch July 7, 2026 21:05
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.

2 participants