Background
lablup/backend.ai currently has no .github/CODEOWNERS file. External and new contributors have no programmatic way to know who reviews changes in a given area, and PRs are not auto-routed to an appropriate reviewer.
A review of the last 50 merged PRs shows that the active human reviewer pool is exactly the @lablup/core_dev team (achimnol, jopemachine, HyeockJinKim, fregataa, seedspirit):
| Reviewer |
Reviews on last 50 PRs |
In @lablup/core_dev |
| HyeockJinKim |
24 |
✓ |
| jopemachine |
13 |
✓ |
| seedspirit |
10 |
✓ |
| fregataa |
6 |
✓ |
| achimnol |
(long-term architect) |
✓ |
So @lablup/core_dev is not a theoretical option — it's a precise codification of who already reviews PRs.
Proposal
Introduce .github/CODEOWNERS as advisory (auto-request reviewers, not branch-protection-required) using @lablup/core_dev as the default owner.
Variant A — team handle (recommended)
Pros:
- Stable across personnel changes (re-member the team, don't edit the file).
- GitHub Code review assignment on
core_dev can round-robin a single reviewer per PR instead of pinging all 5.
- If branch protection is later wired to require CODEOWNERS approval,
@lablup/core_dev is the right gating entity.
Caveats:
- Requires Code review assignment to be enabled on the team — otherwise every PR auto-requests all 5 members. Verify this is set before merging.
- For team-based owners, GitHub requires the team to have at least Read access to the repo (already true:
core_dev has access to 216 repos in the org).
Variant B — explicit individuals
* @achimnol @HyeockJinKim @jopemachine @fregataa @seedspirit
Pros:
- No dependency on team configuration; works identically for org members and external viewers.
- File explicitly names current maintainers (visible without org membership).
Caveats:
- Requires editing the file when membership changes.
- All 5 individuals get auto-requested per PR (no built-in round-robin), increasing notification volume for them — unless we manually pare the list.
Out of scope (clarification)
CODEOWNERS does not replace these existing workflows:
.github/labeler.yml — path-based label application
.github/workflows/* — labeler / size-label / dependabot workflows
Those should remain as-is.
Tasks
Background
lablup/backend.aicurrently has no.github/CODEOWNERSfile. External and new contributors have no programmatic way to know who reviews changes in a given area, and PRs are not auto-routed to an appropriate reviewer.A review of the last 50 merged PRs shows that the active human reviewer pool is exactly the
@lablup/core_devteam (achimnol,jopemachine,HyeockJinKim,fregataa,seedspirit):@lablup/core_devSo
@lablup/core_devis not a theoretical option — it's a precise codification of who already reviews PRs.Proposal
Introduce
.github/CODEOWNERSas advisory (auto-request reviewers, not branch-protection-required) using@lablup/core_devas the default owner.Variant A — team handle (recommended)
Pros:
core_devcan round-robin a single reviewer per PR instead of pinging all 5.@lablup/core_devis the right gating entity.Caveats:
core_devhas access to 216 repos in the org).Variant B — explicit individuals
Pros:
Caveats:
Out of scope (clarification)
CODEOWNERS does not replace these existing workflows:
.github/labeler.yml— path-based label application.github/workflows/*— labeler / size-label / dependabot workflowsThose should remain as-is.
Tasks
@lablup/core_dev.github/CODEOWNERSchanges/src/ai/backend/managerandsrc/ai/backend/commonfirst