From d5ea1776528fa08a3dc384a1d12734ca2b3fabd5 Mon Sep 17 00:00:00 2001
From: Gibson Fahnestock <gibfahn@gmail.com>
Date: Tue, 31 Oct 2017 10:58:07 +0000
Subject: [PATCH] doc: add nodejs/gyp team for GYP related issues

PR-URL: https://github.com/nodejs/node/pull/16638
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
---
 doc/onboarding-extras.md | 75 ++++++++++++++++++++--------------------
 1 file changed, 38 insertions(+), 37 deletions(-)

diff --git a/doc/onboarding-extras.md b/doc/onboarding-extras.md
index 95f5e4050b74a2..bd6350ebfe3ba9 100644
--- a/doc/onboarding-extras.md
+++ b/doc/onboarding-extras.md
@@ -2,43 +2,44 @@
 
 ## Who to CC in issues
 
-| subsystem | maintainers |
-| --- | --- |
-| `benchmark/*` | @nodejs/benchmarking, @mscdex |
-| `bootstrap_node.js` | @fishrock123 |
-| `doc/*`, `*.md` | @nodejs/documentation |
-| `lib/assert` | @nodejs/testing |
-| `lib/buffer` | @nodejs/buffer |
-| `lib/child_process` | @bnoordhuis, @cjihrig |
-| `lib/cluster` | @bnoordhuis, @cjihrig, @mcollina |
-| `lib/{crypto,tls,https}` | @nodejs/crypto |
-| `lib/dgram` | @cjihrig, @mcollina |
-| `lib/domains` | @misterdjules |
-| `lib/fs`, `src/{fs,file}` | @nodejs/fs |
-| `lib/{_}http{*}` | @nodejs/http |
-| `lib/inspector.js`, `src/inspector_*` | @nodejs/v8-inspector |
-| `lib/internal/url`, `src/node_url` | @nodejs/url |
-| `lib/net` | @bnoordhuis, @indutny, @nodejs/streams |
-| `lib/repl` | @addaleax, @fishrock123 |
-| `lib/{_}stream{*}` | @nodejs/streams |
-| `lib/timers` | @fishrock123, @misterdjules |
-| `lib/util` | @bnoordhuis, @cjihrig, @evanlucas |
-| `lib/zlib` | @addaleax, @bnoordhuis, @indutny |
-| `src/async-wrap.*` | @nodejs/async_hooks |
-| `src/node_api.*` | @nodejs/n-api |
-| `src/node_crypto.*` | @nodejs/crypto |
-| `test/*` | @nodejs/testing |
-| `tools/eslint`, `.eslintrc` | @not-an-aardvark, @silverwind, @trott |
-| async_hooks | @nodejs/async_hooks for bugs/reviews (+ @nodejs/diagnostics for API) |
-| build | @nodejs/build |
-| performance | @nodejs/performance |
-| platform specific | @nodejs/platform-{aix,arm,freebsd,macos,ppc,smartos,s390,windows} |
-| python code | @nodejs/python |
-| upgrading c-ares | @jbergstroem |
-| upgrading http-parser | @jbergstroem, @nodejs/http |
-| upgrading libuv | @saghul |
-| upgrading npm | @fishrock123, @MylesBorins |
-| upgrading V8 | @nodejs/v8, @nodejs/post-mortem |
+| Subsystem                             | Maintainers                                                           |
+| ---                                   | ---                                                                   |
+| `benchmark/*`                         | @nodejs/benchmarking, @mscdex                                         |
+| `bootstrap_node.js`                   | @fishrock123                                                          |
+| `doc/*`, `*.md`                       | @nodejs/documentation                                                 |
+| `lib/assert`                          | @nodejs/testing                                                       |
+| `lib/buffer`                          | @nodejs/buffer                                                        |
+| `lib/child_process`                   | @bnoordhuis, @cjihrig                                                 |
+| `lib/cluster`                         | @bnoordhuis, @cjihrig, @mcollina                                      |
+| `lib/{crypto,tls,https}`              | @nodejs/crypto                                                        |
+| `lib/dgram`                           | @cjihrig, @mcollina                                                   |
+| `lib/domains`                         | @misterdjules                                                         |
+| `lib/fs`, `src/{fs,file}`             | @nodejs/fs                                                            |
+| `lib/{_}http{*}`                      | @nodejs/http                                                          |
+| `lib/inspector.js`, `src/inspector_*` | @nodejs/v8-inspector                                                  |
+| `lib/internal/url`, `src/node_url`    | @nodejs/url                                                           |
+| `lib/net`                             | @bnoordhuis, @indutny, @nodejs/streams                                |
+| `lib/repl`                            | @addaleax, @fishrock123                                               |
+| `lib/{_}stream{*}`                    | @nodejs/streams                                                       |
+| `lib/timers`                          | @fishrock123, @misterdjules                                           |
+| `lib/util`                            | @bnoordhuis, @cjihrig, @evanlucas                                     |
+| `lib/zlib`                            | @addaleax, @bnoordhuis, @indutny                                      |
+| `src/async-wrap.*`                    | @nodejs/async\_hooks                                                  |
+| `src/node_api.*`                      | @nodejs/n-api                                                         |
+| `src/node_crypto.*`                   | @nodejs/crypto                                                        |
+| `test/*`                              | @nodejs/testing                                                       |
+| `tools/eslint`, `.eslintrc`           | @not-an-aardvark, @silverwind, @trott                                 |
+| async\_hooks                          | @nodejs/async\_hooks for bugs/reviews (+ @nodejs/diagnostics for API) |
+| build                                 | @nodejs/build                                                         |
+| GYP                                   | @nodejs/gyp                                                           |
+| performance                           | @nodejs/performance                                                   |
+| platform specific                     | @nodejs/platform-{aix,arm,freebsd,macos,ppc,smartos,s390,windows}     |
+| python code                           | @nodejs/python                                                        |
+| upgrading c-ares                      | @jbergstroem                                                          |
+| upgrading http-parser                 | @jbergstroem, @nodejs/http                                            |
+| upgrading libuv                       | @saghul                                                               |
+| upgrading npm                         | @fishrock123, @MylesBorins                                            |
+| upgrading V8                          | @nodejs/v8, @nodejs/post-mortem                                       |
 
 When things need extra attention, are controversial, or `semver-major`: @nodejs/tsc