From c3fbe0209e06fc32270e7eb60c01ee2598575094 Mon Sep 17 00:00:00 2001 From: Aditya Anand Date: Mon, 12 Jun 2017 14:16:19 +0530 Subject: [PATCH 1/2] doc: updated /cc list for inspector issues Updated onboarding-extras.md to /cc @nodejs/v8-inspector for inspector issues. Fixes: https://github.com/nodejs/node/issues/13621 --- doc/onboarding-extras.md | 1 + 1 file changed, 1 insertion(+) diff --git a/doc/onboarding-extras.md b/doc/onboarding-extras.md index 3f174843397b3d..e4e2a580102f9b 100644 --- a/doc/onboarding-extras.md +++ b/doc/onboarding-extras.md @@ -14,6 +14,7 @@ | `lib/dgram` | @cjihrig, @mcollina | | `lib/domains` | @misterdjules | | `lib/fs`, `src/{fs,file}` | @nodejs/fs | +| `lib/inspector.js`, `src/inspector_*` | @nodejs/v8-inspector | | `lib/internal/url`, `src/node_url` | @nodejs/url | | `lib/{_}http{*}` | @nodejs/http | | `lib/net` | @bnoordhuis, @indutny, @nodejs/streams | From 44f2752cc3e7b4feeab88377a87699bf131eb612 Mon Sep 17 00:00:00 2001 From: Aditya Anand Date: Tue, 13 Jun 2017 11:59:02 +0530 Subject: [PATCH 2/2] doc: reorganized /cc list in alphabetical order reorganized /cc list in alphabetical order in onboarding-extras.md Refs: https://github.com/nodejs/node/issues/13621 --- doc/onboarding-extras.md | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/doc/onboarding-extras.md b/doc/onboarding-extras.md index e4e2a580102f9b..2eafe2ffabd1ea 100644 --- a/doc/onboarding-extras.md +++ b/doc/onboarding-extras.md @@ -14,12 +14,12 @@ | `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/{_}http{*}` | @nodejs/http | | `lib/net` | @bnoordhuis, @indutny, @nodejs/streams | -| `lib/{_}stream{*}` | @nodejs/streams | | `lib/repl` | @addaleax, @fishrock123 | +| `lib/{_}stream{*}` | @nodejs/streams | | `lib/timers` | @fishrock123, @misterdjules | | `lib/util` | @bnoordhuis, @cjihrig, @evanlucas | | `lib/zlib` | @addaleax, @bnoordhuis, @indutny | @@ -30,14 +30,13 @@ | `tools/eslint`, `.eslintrc` | @not-an-aardvark, @silverwind, @trott | | async_hooks | @nodejs/async_hooks for bugs/reviews (+ @nodejs/diagnostics for API) | | performance | @nodejs/performance | -| upgrading V8 | @nodejs/v8, @nodejs/post-mortem | -| upgrading npm | @fishrock123, @MylesBorins | +| 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 | -| python code | @nodejs/python | -| platform specific | @nodejs/platform-{aix,arm,freebsd,macos,ppc,smartos,s390,windows} | - +| upgrading npm | @fishrock123, @MylesBorins | +| upgrading V8 | @nodejs/v8, @nodejs/post-mortem | When things need extra attention, are controversial, or `semver-major`: @nodejs/ctc