From 3dc30632755713179f345f4af024bd904c6162d0 Mon Sep 17 00:00:00 2001 From: Michaël Zasso Date: Fri, 22 Dec 2017 16:53:42 +0100 Subject: tools: move eslint from tools to tools/node_modules This is required because we need to add the babel-eslint dependency and it has to be able to resolve "eslint". babel-eslint is required to support future ES features such as async iterators and import.meta. Refs: https://github.com/nodejs/node/pull/17755 PR-URL: https://github.com/nodejs/node/pull/17820 Reviewed-By: Matteo Collina Reviewed-By: Benjamin Gruenbaum --- doc/onboarding-extras.md | 78 ++++++++++++++++++++++++------------------------ 1 file changed, 39 insertions(+), 39 deletions(-) (limited to 'doc/onboarding-extras.md') diff --git a/doc/onboarding-extras.md b/doc/onboarding-extras.md index 9b00b2cecf..4d389376c9 100644 --- a/doc/onboarding-extras.md +++ b/doc/onboarding-extras.md @@ -2,45 +2,45 @@ ## Who to CC in issues -| Subsystem | Maintainers | -| --- | --- | -| `benchmark/*` | @nodejs/benchmarking, @mscdex | -| `bootstrap_node.js` | @fishrock123 | -| `doc/*`, `*.md` | @nodejs/documentation | -| `lib/assert` | @nodejs/testing | -| `lib/async_hooks` | @nodejs/async\_hooks for bugs/reviews (+ @nodejs/diagnostics for API) | -| `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 | -| build | @nodejs/build | -| ES Modules | @bmeck, @Fishrock123, @guybedford, @MylesBorins, @targos | -| 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 | +| Subsystem | Maintainers | +| --- | --- | +| `benchmark/*` | @nodejs/benchmarking, @mscdex | +| `bootstrap_node.js` | @fishrock123 | +| `doc/*`, `*.md` | @nodejs/documentation | +| `lib/assert` | @nodejs/testing | +| `lib/async_hooks` | @nodejs/async\_hooks for bugs/reviews (+ @nodejs/diagnostics for API) | +| `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/node_modules/eslint`, `.eslintrc` | @not-an-aardvark, @silverwind, @trott | +| build | @nodejs/build | +| ES Modules | @bmeck, @Fishrock123, @guybedford, @MylesBorins, @targos | +| 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 -- cgit v1.2.3