Skip to content

chore: remove extensionless MAINTAINERS duplicate - #58

Merged
hyperpolymath merged 1 commit into
mainfrom
chore/dedupe-maintainers
Aug 26, 2026
Merged

chore: remove extensionless MAINTAINERS duplicate#58
hyperpolymath merged 1 commit into
mainfrom
chore/dedupe-maintainers

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Removes the extensionless MAINTAINERS, superseded by MAINTAINERS.adoc.

This isn't only a duplicate — the removed file was wrong. It carried:

  • no SPDX header
  • maintainership attributed to "Metadatastician / @metadatastician"

while MAINTAINERS.adoc carries the CC-BY-SA-4.0 header and correctly names Jonathan D.A. Jewell as Lead Maintainer. It is scaffold boilerplate — byte-identical across 37 of 40 repos that had it, the other 3 trivial variants.

Owner-directed 2026-08-26. Raised as a PR because this repo's ruleset requires pull requests and verified signatures, which a direct API commit cannot satisfy.

🤖 Generated with Claude Code

Scaffold boilerplate superseded by MAINTAINERS.adoc. The removed file
carried no SPDX header and attributed maintainership to
'Metadatastician / @metadatastician'; MAINTAINERS.adoc carries the
CC-BY-SA-4.0 header and names Jonathan D.A. Jewell as Lead Maintainer.

Owner-directed 2026-08-26. Done via PR because these repos' rulesets
require pull requests and verified signatures.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@hyperpolymath
hyperpolymath merged commit a6a9c6a into main Aug 26, 2026
3 checks passed
@hyperpolymath
hyperpolymath deleted the chore/dedupe-maintainers branch August 26, 2026 12:59
@sonarqubecloud

Copy link
Copy Markdown

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: b53de651-758e-4d13-b57d-2abb8d9aa0c9

📥 Commits

Reviewing files that changed from the base of the PR and between d129e6b and bcd78f9.

📒 Files selected for processing (1)
  • MAINTAINERS

📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Removed the maintainer information and contribution guidance document.

Walkthrough

The MAINTAINERS file was deleted. It contained maintainer listings, contribution requirements, responsibilities, expectations, and update metadata.

Changes

Cohort / File(s) Summary
Maintainer documentation removal
MAINTAINERS
Deletes the maintainer and contribution guidance document.

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

Poem

A rabbit checks the tidy tree
The maintainer file is gone
No lists remain among the leaves
The burrow keeps its code
Hop by hop, the patch is done


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.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

@codacy-production codacy-production Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR correctly addresses the redundancy of maintainer documentation by removing the extensionless MAINTAINERS file. This action ensures that MAINTAINERS.adoc remains the single source of truth for maintainer information and licensing requirements. The PR is up to standards, and no blocking issues or gaps in implementation were found.

Test suggestions

  • Verify the file 'MAINTAINERS' is removed from the file system.

TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback

@gitar-bot

gitar-bot Bot commented Aug 26, 2026

Copy link
Copy Markdown

Important

You are using the Gitar free plan. Upgrade to unlock code review, CI analysis, auto-apply, custom automations, and more.

Gitar

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