Skip to content

Add CODEOWNERS file#74

Merged
starbops merged 2 commits into
harvester:mainfrom
Vicente-Cheng:add-codeowners
Jul 16, 2026
Merged

Add CODEOWNERS file#74
starbops merged 2 commits into
harvester:mainfrom
Vicente-Cheng:add-codeowners

Conversation

@Vicente-Cheng

Copy link
Copy Markdown
Contributor

Problem:

Add CODEOWNER

Solution:

Add CODEOWNER

Related Issue(s):

harvester/harvester#9607

Test plan:

Additional documentation or context

Assign code owners following the org-wide convention: top 3
contributors of the last 24 months (>=2 commits, bots excluded)
with @harvester/dev as fallback.

Signed-off-by: Vicente Cheng <[email protected]>
Copilot AI review requested due to automatic review settings July 13, 2026 09:24

Copilot AI 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

Adds a repository-wide CODEOWNERS file so GitHub can automatically request reviews from the designated owners.

Changes:

  • Introduces a new CODEOWNERS file at the repository root
  • Sets default ownership for all paths to @starbops and @harvester/dev

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread CODEOWNERS Outdated
Comment on lines +3 to +8
# We setup the codeowners with the following rules:
# Top 3 contributors of each directory are assigned as code owners, if don't have enough contributors, dev team will be assigned.
# If the number of commits of a contributor is less than 2, that contributor will be ignored.

# The corresponding git commit is used to determine the code owners.
# git log --since="24 months ago" --pretty=format:'%an' -- <path> | sort | uniq -c | sort -nr | head -n 3
Updated the CODEOWNERS file to clarify ownership rules and formatting.

Signed-off-by: freeze <[email protected]>
@Vicente-Cheng
Vicente-Cheng requested a review from starbops July 14, 2026 01:59

@starbops starbops left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thank you.

@starbops
starbops merged commit c57436a into harvester:main Jul 16, 2026
12 checks passed
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.

3 participants