Commit Graph

6 Commits

Author SHA1 Message Date
Andrea Luzzardi
05a6e78856
Merge pull request #2157 from shykes/universe-x
chore: separate review rotation on universe.dagger.io/x/
2022-04-13 11:27:54 -07:00
Solomon Hykes
49861839ab chore: explicit codeowners for CLI
Signed-off-by: Solomon Hykes <solomon@dagger.io>
2022-04-13 13:01:15 -05:00
Solomon Hykes
778c55a541 chore: separate review rotation on universe.dagger.io/x/
Signed-off-by: Solomon Hykes <solomon@dagger.io>
2022-04-12 18:09:30 -05:00
Solomon Hykes
f57fb98a5b chore: explicit codeowners for the CODEOWNERS file
Signed-off-by: Solomon Hykes <solomon@dagger.io>
2022-04-12 17:52:24 -05:00
Andrea Luzzardi
9547d495b9 ci: Use default CODEOWNERS for docs sidebar
Signed-off-by: Andrea Luzzardi <aluzzardi@gmail.com>
2022-04-07 12:34:27 -07:00
Andrea Luzzardi
e50a61d637 ci: add CODEOWNERS
CODEOWNERS is a tool to encode PR approval rules.

When a PR is opened, at least one code owner is required to approve it
before being merged.

It does **not**:

- Limit reviewers: Everyone is welcome and encouraged to review any PR.
  But at least one CODEOWNER must approve before merging.
- Limit contributions or ownership: Every maintainer is responsible for
  the entire project. CODEOWNERs are there to review PRs for
  consistency.

By default, any maintainer can approve any PR. There's a couple of
exceptions for consistency/specialty:

- Core API: The core API must be consistent and changes are hard
  to reverse
- Universe API: Similar to the Core API, it must be kept consistent. In
  the future, we will have specialized code owners (e.g. universe/aws)
- Docs website: Requires specialized (frontend) skills to maintain for
  visual consistency.

Signed-off-by: Andrea Luzzardi <aluzzardi@gmail.com>
2022-03-31 14:40:57 -07:00