Skip to content

Commit d5ea177

Browse files
gibfahncjihrig
authored andcommitted
doc: add nodejs/gyp team for GYP related issues
PR-URL: #16638 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Yuta Hiroto <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Joyee Cheung <[email protected]>
1 parent 3241799 commit d5ea177

File tree

1 file changed

+38
-37
lines changed

1 file changed

+38
-37
lines changed

‎doc/onboarding-extras.md‎

Lines changed: 38 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -2,43 +2,44 @@
22

33
## Who to CC in issues
44

5-
| subsystem | maintainers |
6-
| --- | --- |
7-
|`benchmark/*`|@nodejs/benchmarking, @mscdex|
8-
|`bootstrap_node.js`|@fishrock123|
9-
|`doc/*`, `*.md`|@nodejs/documentation|
10-
|`lib/assert`|@nodejs/testing|
11-
|`lib/buffer`|@nodejs/buffer|
12-
|`lib/child_process`|@bnoordhuis, @cjihrig|
13-
|`lib/cluster`|@bnoordhuis, @cjihrig, @mcollina|
14-
|`lib/{crypto,tls,https}`|@nodejs/crypto|
15-
|`lib/dgram`|@cjihrig, @mcollina|
16-
|`lib/domains`|@misterdjules|
17-
|`lib/fs`, `src/{fs,file}`|@nodejs/fs|
18-
|`lib/{_}http{*}`|@nodejs/http|
19-
|`lib/inspector.js`, `src/inspector_*`|@nodejs/v8-inspector|
20-
|`lib/internal/url`, `src/node_url`|@nodejs/url|
21-
|`lib/net`|@bnoordhuis, @indutny, @nodejs/streams|
22-
|`lib/repl`|@addaleax, @fishrock123|
23-
|`lib/{_}stream{*}`|@nodejs/streams|
24-
|`lib/timers`|@fishrock123, @misterdjules|
25-
|`lib/util`|@bnoordhuis, @cjihrig, @evanlucas|
26-
|`lib/zlib`|@addaleax, @bnoordhuis, @indutny|
27-
|`src/async-wrap.*`|@nodejs/async_hooks |
28-
|`src/node_api.*`|@nodejs/n-api|
29-
|`src/node_crypto.*`|@nodejs/crypto|
30-
|`test/*`|@nodejs/testing|
31-
|`tools/eslint`, `.eslintrc`|@not-an-aardvark, @silverwind, @trott|
32-
| async_hooks |@nodejs/async_hooks for bugs/reviews (+ @nodejs/diagnostics for API) |
33-
| build |@nodejs/build|
34-
| performance |@nodejs/performance|
35-
| platform specific |@nodejs/platform-{aix,arm,freebsd,macos,ppc,smartos,s390,windows} |
36-
| python code |@nodejs/python|
37-
| upgrading c-ares |@jbergstroem|
38-
| upgrading http-parser |@jbergstroem, @nodejs/http|
39-
| upgrading libuv |@saghul|
40-
| upgrading npm |@fishrock123, @MylesBorins|
41-
| upgrading V8 |@nodejs/v8, @nodejs/post-mortem|
5+
| Subsystem | Maintainers |
6+
| --- | --- |
7+
|`benchmark/*`|@nodejs/benchmarking, @mscdex|
8+
|`bootstrap_node.js`|@fishrock123|
9+
|`doc/*`, `*.md`|@nodejs/documentation|
10+
|`lib/assert`|@nodejs/testing|
11+
|`lib/buffer`|@nodejs/buffer|
12+
|`lib/child_process`|@bnoordhuis, @cjihrig|
13+
|`lib/cluster`|@bnoordhuis, @cjihrig, @mcollina|
14+
|`lib/{crypto,tls,https}`|@nodejs/crypto|
15+
|`lib/dgram`|@cjihrig, @mcollina|
16+
|`lib/domains`|@misterdjules|
17+
|`lib/fs`, `src/{fs,file}`|@nodejs/fs|
18+
|`lib/{_}http{*}`|@nodejs/http|
19+
|`lib/inspector.js`, `src/inspector_*`|@nodejs/v8-inspector|
20+
|`lib/internal/url`, `src/node_url`|@nodejs/url|
21+
|`lib/net`|@bnoordhuis, @indutny, @nodejs/streams|
22+
|`lib/repl`|@addaleax, @fishrock123|
23+
|`lib/{_}stream{*}`|@nodejs/streams|
24+
|`lib/timers`|@fishrock123, @misterdjules|
25+
|`lib/util`|@bnoordhuis, @cjihrig, @evanlucas|
26+
|`lib/zlib`|@addaleax, @bnoordhuis, @indutny|
27+
|`src/async-wrap.*`|@nodejs/async\_hooks |
28+
|`src/node_api.*`|@nodejs/n-api|
29+
|`src/node_crypto.*`|@nodejs/crypto|
30+
|`test/*`|@nodejs/testing|
31+
|`tools/eslint`, `.eslintrc`|@not-an-aardvark, @silverwind, @trott|
32+
| async\_hooks |@nodejs/async\_hooks for bugs/reviews (+ @nodejs/diagnostics for API) |
33+
| build |@nodejs/build|
34+
| GYP |@nodejs/gyp|
35+
| performance |@nodejs/performance|
36+
| platform specific |@nodejs/platform-{aix,arm,freebsd,macos,ppc,smartos,s390,windows} |
37+
| python code |@nodejs/python|
38+
| upgrading c-ares |@jbergstroem|
39+
| upgrading http-parser |@jbergstroem, @nodejs/http|
40+
| upgrading libuv |@saghul|
41+
| upgrading npm |@fishrock123, @MylesBorins|
42+
| upgrading V8 |@nodejs/v8, @nodejs/post-mortem|
4243

4344
When things need extra attention, are controversial, or `semver-major`: @nodejs/tsc
4445

0 commit comments

Comments
(0)