You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
To ensure the right people approve changes to certain files/folders, this PR does the following:
Adds ownership of design system-related changes to a newly created design-system team
Same for petrinaut
Assigns existing backend team as owner of the Node API and core Graph packages.
It also fixes existing issues:
LICENSE matching: the existing patterns missed e.g. LICENSE-MIT.md. The PR replaces the existing duplicative patterns with LICENSE* (matching is case sensitive so this won't match e.g. license.svg).
The codeowner logic is 'last matched wins', so previously any license files in e.g. /libs/@local/graph/postgres-store/postgres_migrations/ would have not needed legal approval (as the legal ownership was overridden by the later backend ownership). The PR moves the most specific rules (e.g. turbo.json) to the bottom.
Low Risk
Only GitHub review routing in CODEOWNERS changes; no application or runtime behavior.
Overview
Expands .github/CODEOWNERS so review routing matches major areas of the repo: Rust (*.rs, Cargo.toml, hashql/harpc), design system (hashdotdesign and ds-* libs), Petrinaut apps/libs, and backend (hash-api, hash-graph, @local/graph), with migration paths still owned by backend.
Legal coverage now uses a single LICENSE* pattern (so variants like LICENSE-MIT.md are included) instead of three fixed filenames. Rules are reordered so broader path rules (e.g. migrations, turbo, env) no longer override more specific ones—especially legal on license files under paths that also have backend owners, since GitHub applies last match wins.
Reviewed by Cursor Bugbot for commit 880a879. Bugbot is set up for automated code reviews on this repo. Configure here.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
area/infraRelates to version control, CI, CD or IaC (area)
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
github-actions
Bot
added
the
area/infra
🌟 Purpose
To ensure the right people approve changes to certain files/folders, this PR does the following:
design-systemteampetrinautbackendteam as owner of the Node API and core Graph packages.It also fixes existing issues:
LICENSE-MIT.md. The PR replaces the existing duplicative patterns with LICENSE* (matching is case sensitive so this won't match e.g.license.svg)./libs/@local/graph/postgres-store/postgres_migrations/would have not needed legal approval (as the legal ownership was overridden by the later backend ownership). The PR moves the most specific rules (e.g.turbo.json) to the bottom.