Update codeowners to use Vercel Spaces (#50841)

Move as much of codeowners as possible to use Vercel Spaces. 

1. Makes `@timneutkens @ijjk @shuding @huozhi @feedthejim` global owners
2. Make the `@vercel/next-js` team _optional_ owners of **/docs**,
**/errors**, and **/contributing**, makes team owners of a few packages
as per old config.
3. Make `@vercel/devex` (docs and devrel) owners of **/docs**,
**/errors**, and **/contributing**
4. Make `@vercel/devrel` (devrel only) owners of **/examples**
5. Make `@vercel/web-tooling` owners of specific files and folders (as
per old config)

Leaves @styfle as owner of **image** files on the old config since this
pattern `/**/*image*/** ` can't be used with Vercel Spaces.

Note: We cannot add * or / at the end of files.
[Docs](https://spaces-docs.vercel.sh/docs/code-owners#:~:text=Code%20Owners%20files%20are%20meant%20to%20encourage%20distributed%20ownership%20definitions%20across%20a%20codebase.)
This commit is contained in:
Delba de Oliveira 2023-06-08 11:02:18 +01:00 committed by GitHub
parent f6ff2ef98e
commit 8c2e5bd072
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
11 changed files with 43 additions and 18 deletions

21
.github/CODEOWNERS vendored
View file

@ -1,18 +1,13 @@
# Learn how to add code owners here:
# https://help.github.com/en/articles/about-code-owners
* @timneutkens @ijjk @shuding @huozhi @feedthejim
# Part of code owners now use Vercel Spaces
# Search .vercel.approvers for all files
/.git* @vercel/next-js
/docs/ @vercel/next-js @leerob
/errors/ @vercel/next-js @leerob
/examples/ @vercel/next-js @leerob @steven-tey
/scripts/ @vercel/next-js
/.alex* @vercel/next-js @leerob
/.eslint* @vercel/next-js @leerob
/.prettier* @vercel/next-js @leerob
/*.md @vercel/next-js @leerob
/packages/create-next-app/ @vercel/next-js
/pnpm-lock.yaml @vercel/next-js @vercel/web-tooling
# Image Component (@styfle)
@ -22,13 +17,3 @@
/packages/next/server/lib/squoosh/ @timneutkens @ijjk @shuding @styfle
/packages/next/server/serve-static.ts @timneutkens @ijjk @shuding @styfle @huozhi
/packages/next/server/config.ts @timneutkens @ijjk @shuding @styfle @huozhi
# Tooling & Telemetry
/packages/next/src/build/ @timneutkens @ijjk @shuding @vercel/web-tooling @huozhi
/packages/next/src/telemetry/ @timneutkens @ijjk @shuding @padmaia
/packages/next-swc/ @timneutkens @ijjk @shuding @vercel/web-tooling
Cargo.toml @timneutkens @ijjk @shuding @vercel/web-tooling
Cargo.lock @timneutkens @ijjk @shuding @vercel/web-tooling
/.cargo/config.toml @timneutkens @ijjk @shuding @vercel/web-tooling
/.config/nextest.toml @timneutkens @ijjk @shuding @vercel/web-tooling

17
.vercel.approvers Normal file
View file

@ -0,0 +1,17 @@
# Global Owners
@timneutkens
@ijjk
@shuding
@huozhi
@feedthejim
# Tooling & Telemetry
pnpm-lock.yaml @vercel/web-tooling
Cargo.toml @vercel/web-tooling
Cargo.lock @vercel/web-tooling
/.cargo/config.toml @vercel/web-tooling
/.config/nextest.toml @vercel/web-tooling
# Catch any markdown files outside docs (e.g. examples)
**/*.md @vercel/devex
**/*.mdx @vercel/devex

View file

@ -0,0 +1,2 @@
@vercel/devex
@vercel/next-js:optional

2
docs/.vercel.approvers Normal file
View file

@ -0,0 +1,2 @@
@vercel/devex
@vercel/next-js:optional

2
errors/.vercel.approvers Normal file
View file

@ -0,0 +1,2 @@
@vercel/devex
@vercel/next-js:optional

View file

@ -0,0 +1,2 @@
@vercel/devrel
@vercel/next-js:optional

View file

@ -0,0 +1 @@
@vercel/next-js

View file

@ -0,0 +1,4 @@
@vercel/web-tooling
@timneutkens
@ijjk
@shuding

View file

@ -0,0 +1,5 @@
@vercel/web-tooling
@timneutkens
@ijjk
@shuding
@huozhi

View file

@ -0,0 +1,4 @@
@padmaia
@timneutkens
@ijjk
@shuding

View file

@ -0,0 +1 @@
@vercel/next-js