Skip to content

Update CODEOWNERS - #990

Merged
turbolent merged 1 commit into
mainfrom
bastian/codeowners-flow-core-protocol
Aug 20, 2026
Merged

Update CODEOWNERS#990
turbolent merged 1 commit into
mainfrom
bastian/codeowners-flow-core-protocol

Conversation

@turbolent

@turbolent turbolent commented Aug 20, 2026

Copy link
Copy Markdown
Member

Consolidate code ownership onto the @onflow/flow-core-protocol team.

Summary by CodeRabbit

  • Chores
    • Updated repository ownership configuration to reflect the current designated maintainers.

@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: a2fd85b3-341d-4b9e-a041-e22f37da5f43

📥 Commits

Reviewing files that changed from the base of the PR and between da1e41e and 567f855.

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

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The repository-wide CODEOWNERS rule now assigns ownership to flow-core-protocol instead of four individual maintainers.

Changes

Ownership Update

Layer / File(s) Summary
Repository ownership assignment
.github/CODEOWNERS
The wildcard CODEOWNERS entry now assigns ownership to flow-core-protocol.

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

Merge Risk: ⚪ Minimal · up to 567f8

This change only consolidates repository ownership onto the specified team and introduces no actionable merge-blocking risk beyond normal review.

Suggested reviewers: zhangchiqing, m-peter, janezpodhostnik, peterargue

🚥 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 clearly identifies the main change: updating the CODEOWNERS file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bastian/codeowners-flow-core-protocol

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.

@turbolent
turbolent merged commit 4513576 into main Aug 20, 2026
2 checks passed
@turbolent
turbolent deleted the bastian/codeowners-flow-core-protocol branch August 20, 2026 19:51
@github-project-automation github-project-automation Bot moved this from 👀 In Review to ✅ Done in 🌊 Flow 4D Aug 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants