Skip to content

chore: revise CODEOWNERS file for ownership adjustments#285

Open
hendrikebbers wants to merge 2 commits intomainfrom
hendrikebbers-patch-5
Open

chore: revise CODEOWNERS file for ownership adjustments#285
hendrikebbers wants to merge 2 commits intomainfrom
hendrikebbers-patch-5

Conversation

@hendrikebbers
Copy link
Copy Markdown
Member

@hendrikebbers hendrikebbers commented Mar 17, 2026

Updated CODEOWNERS file to modify ownership rules and simplify permissions for various files.

Summary by CodeRabbit

  • Chores
    • Simplified repository ownership rules by removing a global catch-all and redundant entries, consolidating targeted ownership for specific project files, narrowing maintainers for certain root and config files, and preserving protection for key repository root documents.

Updated CODEOWNERS file to modify ownership rules and simplify permissions for various files.

Signed-off-by: Hendrik Ebbers <hendrik.ebbers@web.de>
@hendrikebbers hendrikebbers requested a review from a team as a code owner March 17, 2026 15:31
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 17, 2026

Deploy Preview for hiero-open-source ready!

Name Link
🔨 Latest commit c9afe05
🔍 Latest deploy log https://app.netlify.com/projects/hiero-open-source/deploys/69d62c40c037fa000865373a
😎 Deploy Preview https://deploy-preview-285--hiero-open-source.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ec249020-a533-4343-b780-db8123b78eac

📥 Commits

Reviewing files that changed from the base of the PR and between 71be7d2 and c9afe05.

📒 Files selected for processing (1)
  • .github/CODEOWNERS
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/CODEOWNERS

📝 Walkthrough

Walkthrough

Updated .github/CODEOWNERS to remove broad catch‑all and overlapping rules, consolidating ownership into targeted path-specific entries for .github/, package.json, package-lock.json, .remarkrc, CODEOWNERS, README.md, LICENSE, and .gitignore patterns.

Changes

Cohort / File(s) Summary
CODEOWNERS Configuration
\.github/CODEOWNERS
Removed global catch‑all and overlapping group entries; added/retained targeted ownership for /.github/, package.json, package-lock.json, .remarkrc, /CODEOWNERS, /README.md, **/LICENSE, and narrowed ownership for **/codecov.yml and **/.gitignore*.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • chore: audit #244 — Also updates .github/CODEOWNERS, adjusting overlapping and global ownership rules.

Poem

🐰
In the burrow of repos I tidy the trails,
Snipping broad vines into neat little rails.
Each file now knows who guards day and night,
No overlaps muddle the keepers’ sight.
Hooray for clean rules — a hop and delight! 🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is minimal and lacks required template sections such as detailed Changes Made, Related Issues, Checklist items, and Deployment Notes. Expand the description to follow the repository template with detailed Changes Made section, Related Issues references, and completed Checklist items for better clarity.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: revising the CODEOWNERS file for ownership adjustments, which matches the core purpose of the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hendrikebbers-patch-5

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 and usage tips.

@lfdt-bot
Copy link
Copy Markdown
Contributor

lfdt-bot commented Mar 17, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

jwagantall
jwagantall previously approved these changes Mar 17, 2026
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/CODEOWNERS:
- Around line 3-6: Update the CODEOWNERS entries so the listed filenames are
anchored to the repository root by prefixing them with a leading slash: change
package.json, package-lock.json and .remarkrc to /package.json,
/package-lock.json and /.remarkrc respectively to avoid matching those names in
nested directories; ensure the ownership tokens (e.g.,
`@hiero-ledger/hiero-website-maintainers` and `@hiero-ledger/github-maintainers`)
remain unchanged.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 632c2e87-7c01-4398-89cc-227d170091dc

📥 Commits

Reviewing files that changed from the base of the PR and between 7f14ab9 and 71be7d2.

📒 Files selected for processing (1)
  • .github/CODEOWNERS

@jwagantall jwagantall changed the title Revise CODEOWNERS file for ownership adjustments chore: revise CODEOWNERS file for ownership adjustments Mar 23, 2026
Co-authored-by: Nathan Klick <nathan@swirldslabs.com>
Signed-off-by: exploreriii <133720349+exploreriii@users.noreply.github.com>
@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

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

@exploreriii
Copy link
Copy Markdown
Contributor

@hendrikebbers I applied the suggestion for the codeowners, is there any other changes you'd like to make or is this ready to review again?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants