Skip to content

docs(#631): de-duplicate versioning.md (.github-private is authoritative)#632

Merged
don-petry merged 1 commit into
mainfrom
fix-631-dedup-versioning
Jul 9, 2026
Merged

docs(#631): de-duplicate versioning.md (.github-private is authoritative)#632
don-petry merged 1 commit into
mainfrom
fix-631-dedup-versioning

Conversation

@don-petry

@don-petry don-petry commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Closes #631. Follow-up to epic #613.

What

Epic #613 (#624) copied docs/release/versioning.md into .github, creating a duplicate that has since diverged from the .github-private original.

.github's own standards already name .github-private as authoritative (ci-standards.md; agent-canary-rings-adr.md links there), and the doc covers cut-release.sh which lives in .github-private.

Change

No content lost. No broken relative links remain (verified). markdownlint clean.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated release-notes cross-references to point to the current online versioning documentation.
    • Removed an outdated release/versioning guide from the documentation set.

…ub-private canonical

Epic #613 (PR #624) copied docs/release/versioning.md into .github, creating a second
copy that has since diverged from the .github-private original. .github's own standards
already name .github-private as authoritative (ci-standards.md §"authoritative process
lives in that repo's docs/release/"; agent-canary-rings-adr.md links there), and the doc
documents cut-release.sh which lives in .github-private.

- Remove docs/release/versioning.md (the redundant #624 copy).
- Repoint canary-rollout-adr.md's two links to the .github-private canonical URL.

No content lost — canonical copy stays in .github-private.

Closes #631. Refs #613

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@don-petry
don-petry requested a review from a team as a code owner July 9, 2026 18:50
Copilot AI review requested due to automatic review settings July 9, 2026 18:50
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 23ce104f-7e80-4493-a4d6-667bd8fe4186

📥 Commits

Reviewing files that changed from the base of the PR and between 10241b4 and dbb5375.

📒 Files selected for processing (2)
  • docs/initiatives/canary-rollout-adr.md
  • docs/release/versioning.md
💤 Files with no reviewable changes (1)
  • docs/release/versioning.md

📝 Walkthrough

Walkthrough

This PR removes the duplicated docs/release/versioning.md file and updates two references in docs/initiatives/canary-rollout-adr.md to point to the canonical .github-private absolute URL instead of the previous relative path, reformatting surrounding text accordingly.

Changes

Versioning doc de-duplication

Layer / File(s) Summary
Repoint references and remove duplicate
docs/initiatives/canary-rollout-adr.md, docs/release/versioning.md
The Context and References sections' links to versioning.md are changed from relative paths to absolute .github-private GitHub URLs with reformatted text, and the duplicated versioning.md file is deleted entirely.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

  • petry-projects/.github#624: Original PR that added the canary-rollout-adr.md reference and the duplicated docs/release/versioning.md file now being cleaned up.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing the duplicate versioning doc and keeping the .github-private copy authoritative.
Linked Issues check ✅ Passed The PR deletes the redundant versioning.md copy and repoints the two ADR links to the canonical .github-private URL as requested.
Out of Scope Changes check ✅ Passed All changes relate to de-duplicating versioning docs and updating references; no unrelated edits are evident.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-631-dedup-versioning

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

sonarqubecloud Bot commented Jul 9, 2026

Copy link
Copy Markdown

@don-petry

Copy link
Copy Markdown
Contributor Author

Dev-Lead — waiting on PR blockers (intent: review-changes)

PR: #632
No changes were committed, but the PR still has blocking checks or reviews (failing or cancelled checks, or changes-requested reviews). The retry cron will re-attempt automatically. Next attempt after: 2026-07-09T19:21:04Z

@don-petry

Copy link
Copy Markdown
Contributor Author

Note

@don-petry I reviewed this PR and no code changes were needed, but it still has blocking checks or reviews (failing or cancelled checks, or changes-requested reviews), so I cannot mark it done yet. I'll re-check automatically.
Next attempt after: 2026-07-09T19:21:04Z

@don-petry
don-petry enabled auto-merge (squash) July 9, 2026 18:51
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

Gemini encountered an error creating the review. You can try again by commenting /gemini review.

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

Removes a duplicated versioning.md document from this repo and updates references to point at the canonical .github-private copy, avoiding ongoing divergence while keeping the documentation source-of-truth co-located with the tooling it describes.

Changes:

  • Deleted docs/release/versioning.md (redundant, diverging copy).
  • Updated docs/initiatives/canary-rollout-adr.md to link to the canonical .github-private versioning.md URL in both referenced locations.

Reviewed changes

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

File Description
docs/release/versioning.md Removes the redundant local copy to prevent further divergence from the canonical doc in .github-private.
docs/initiatives/canary-rollout-adr.md Repoints references from the removed relative doc to the canonical .github-private URL.

@don-petry
don-petry disabled auto-merge July 9, 2026 18:52
@don-petry

Copy link
Copy Markdown
Contributor Author

Dev-Lead — fix-bot-comment (no-changes)

Agent reasoning
Issues addressed: 0
- Bot encountered an error generating review; no specific findings provided
Files changed: 0
Tier 1 blockers: 0
```
**Analysis:**
The bot comment is an error from the Gemini reviewer—it failed to generate a review and doesn't contain any specific code findings. Looking at the actual PR state:
1. **CI status:** All checks are passing or in progress. No failures, timeouts, or action-required states.
2. **Reviews:** Only the `copilot-pull-request-reviewer[bot]` has reviewed (state: `COMMENTED`, no changes requested). It reviewed both changed files and generated no comments.
3. **Tier 1 blockers:** None (no failed checks, no `CHANGES_REQUESTED` reviews).
**PR changes (verified):**
- Deleted `docs/release/versioning.md` (redundant copy)
- Updated `docs/initiatives/canary-rollout-adr.md` (two references redirected to `.github-private` canonical URL)
Since the bot provided no actionable code findings, only an error message about its own failure, and all other checks pass with no reviewer objections, **no code changes are needed**. The PR is ready.

@don-petry
don-petry enabled auto-merge (squash) July 9, 2026 18:52
@don-petry
don-petry merged commit 8563923 into main Jul 9, 2026
26 of 28 checks passed
@don-petry
don-petry deleted the fix-631-dedup-versioning branch July 9, 2026 18:54
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.

docs(#613 follow-up): de-duplicate versioning.md — .github-private is authoritative

2 participants