Skip to content

Remove stale metadata from contributor documentation - #52

Closed
coderabbitai[bot] wants to merge 2 commits into
mainfrom
coderabbit/fix-ci-governance-checks/96c22b0d
Closed

coderabbitai[bot] wants to merge 2 commits into
mainfrom
coderabbit/fix-ci-governance-checks/96c22b0d

Conversation

@coderabbitai

@coderabbitai coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Remove outdated version, last-updated, and next-review metadata from CONTRIBUTING.adoc and REVERSIBILITY.adoc to satisfy CI governance checks.

Validation: git diff --check passed.

View coding task

hyperpolymath and others added 2 commits September 9, 2026 02:18
This repo's standards reusable pins are re-pointed at the standards default-branch HEAD, resolved live at sweep time. The prior refs are recorded in the verification line below. Three kinds of drift are repaired together and the body does not claim which one this repo had: an UNREACHABLE sha kills the run at workflow STARTUP, so GitHub reports no check at all rather than a failing one and the gate disappears instead of going red; a FLOATING ref (@main) is unpinned supply chain; a merely STALE but reachable sha silently reintroduces every bug fixed since it. files=4 pins=4 perms=1 permlines=1 from=092dedada188f56c5915f74a5fd40aac093742c3 target=8f2ee508

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0178nN4Nm3neFRy5K9StZKnB
@coderabbitai

coderabbitai Bot commented Sep 9, 2026 •

Copy link
Copy Markdown
Contributor Author

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Essentials

Run ID: 776b172d-f279-4096-9539-0414c1f44c61

📥 Commits

Reviewing files that changed from the base of the PR and between 8f37719 and 56a3d37.

📒 Files selected for processing (2)
  • CONTRIBUTING.adoc
  • REVERSIBILITY.adoc
💤 Files with no reviewable changes (2)
  • CONTRIBUTING.adoc
  • REVERSIBILITY.adoc

Included review availability: 4 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


📝 Summary

Summary by CodeRabbit

  • Documentation
    • Removed footer metadata from the contribution guidelines, including the last-updated date and version number.
    • Removed version, last-updated date, and next-review date metadata from the reversibility guidance.
    • The existing reversibility guidance remains unchanged.

Walkthrough

The change removes version and date metadata from CONTRIBUTING.adoc and REVERSIBILITY.adoc. The remaining document guidance is unchanged.

Changes

Documentation metadata

Layer / File(s) Summary
Remove document metadata
CONTRIBUTING.adoc, REVERSIBILITY.adoc
Removed footer and metadata lines that contained version, update-date, and review-date information.

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

Suggested reviewers: hyperpolymath

Merge Risk: ⚪ Minimal · up to 56a3d

The change is limited to removing obsolete document metadata, with no remaining merge-blocking risk identified.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly summarises the removal of stale metadata from the contributor documentation. It is concise and directly related to the main changes.
Description check ✅ Passed The description accurately explains the removal of outdated metadata from CONTRIBUTING.adoc and REVERSIBILITY.adoc. It also records the validation result.

A rabbit checks the page at dawn
The dated footer hops away
Version marks are gone
The guidance stays in place
Clean documents greet the day

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

@hyperpolymath
hyperpolymath added this pull request to stack #54 September 12, 2026 10:26
hyperpolymath
hyperpolymath previously approved these changes Sep 12, 2026

@hyperpolymath hyperpolymath left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving CodeRabbit PR: Remove stale metadata from contributor documentation.

@hyperpolymath

Copy link
Copy Markdown
Owner

🟢 READY FOR MANUAL MERGE

✅ All checks passing (commit status: success)
✅ 1 approval from hyperpolymath
✅ No CodeRabbit reviews/recommendations
✅ Clean state

📋 Action: This PR is part of stack PRS_kwDOS7euK84AEGOx. GitHub's REST API cannot merge stacked PRs (async endpoint returns 404). Please merge manually via GitHub web interface.

@hyperpolymath

Copy link
Copy Markdown
Owner

🟡 READY FOR MANUAL MERGE + NEEDS 2ND APPROVAL

✅ CodeRabbit PR approved by hyperpolymath (1 approval)
✅ All checks passing (commit status: success)
✅ No CodeRabbit reviews/recommendations

⚠️ Blocked by TWO issues:

  1. Stacked PR: Part of stack PRS_kwDOS7euK84AEGOx - REST API returns 404
  2. Ruleset: Optimus-Branch requires 2 approvals (current: 1)

📋 Action:

  • Add 1 more approval from any user with write access
  • THEN merge manually via GitHub web interface (due to stack)

Base automatically changed from fix/bump-diverged-standards-pin to main September 13, 2026 23:22
@coderabbitai
coderabbitai Bot dismissed hyperpolymath’s stale review September 13, 2026 23:22

The merge-base changed after approval.

@hyperpolymath

Copy link
Copy Markdown
Owner

Closing as superseded: main already removed the stale Last updated/Version metadata block from CONTRIBUTING.adoc (and the workflow pin bumps this branch carried are identical to main's). The only remaining delta after merging main was the deletion of the trailing --- separator, which main deliberately keeps.

@hyperpolymath
hyperpolymath deleted the coderabbit/fix-ci-governance-checks/96c22b0d branch September 18, 2026 13:12
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