diff options
author | Michael Dawson <mdawson@devrus.com> | 2022-01-05 21:26:30 +0300 |
---|---|---|
committer | Michael Dawson <mdawson@devrus.com> | 2022-01-18 22:24:30 +0300 |
commit | a199387f045aa55b58c6d2e52bb69bf32c9d3d5e (patch) | |
tree | e7570b05d54b2a73edc9b7c2481a5730385654d2 | |
parent | 65910c0d6cdbca6d8efa919378d72eff34016f35 (diff) |
doc: make contributing info more discoverable
There are been several discussions in recent PRs about
the docs related to contributing not being very discoverable.
Move these docs from doc/guides/ to doc/contributing.
Signed-off-by: Michael Dawson <mdawson@devrus.com>
PR-URL: https://github.com/nodejs/node/pull/41408
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Michaƫl Zasso <targos@protonmail.com>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
Reviewed-By: James M Snell <jasnell@gmail.com>
-rw-r--r-- | .github/CODEOWNERS | 6 | ||||
-rw-r--r-- | .github/PULL_REQUEST_TEMPLATE.md | 2 | ||||
-rw-r--r-- | .github/workflows/auto-start-ci.yml | 2 | ||||
-rw-r--r-- | CONTRIBUTING.md | 26 | ||||
-rw-r--r-- | GOVERNANCE.md | 4 | ||||
-rw-r--r-- | Makefile | 4 | ||||
-rw-r--r-- | README.md | 6 | ||||
-rw-r--r-- | benchmark/README.md | 8 | ||||
-rw-r--r-- | deps/openssl/README.md | 2 | ||||
-rw-r--r-- | doc/contributing/adding-new-napi-api.md (renamed from doc/guides/adding-new-napi-api.md) | 0 | ||||
-rw-r--r-- | doc/contributing/backporting-to-release-lines.md (renamed from doc/guides/backporting-to-release-lines.md) | 0 | ||||
-rw-r--r-- | doc/contributing/building-node-with-ninja.md (renamed from doc/guides/building-node-with-ninja.md) | 0 | ||||
-rw-r--r-- | doc/contributing/code-of-conduct.md (renamed from doc/guides/contributing/code-of-conduct.md) | 0 | ||||
-rw-r--r-- | doc/contributing/collaborator-guide.md (renamed from doc/guides/collaborator-guide.md) | 6 | ||||
-rw-r--r-- | doc/contributing/commit-queue.md (renamed from doc/guides/commit-queue.md) | 0 | ||||
-rw-r--r-- | doc/contributing/components-in-core.md (renamed from doc/guides/components-in-core.md) | 0 | ||||
-rw-r--r-- | doc/contributing/cpp-style-guide.md (renamed from doc/guides/cpp-style-guide.md) | 2 | ||||
-rw-r--r-- | doc/contributing/diagnostic-tooling-support-tiers.md (renamed from doc/guides/diagnostic-tooling-support-tiers.md) | 0 | ||||
-rw-r--r-- | doc/contributing/doc_img/compare-boxplot.png (renamed from doc/guides/doc_img/compare-boxplot.png) | bin | 266641 -> 266641 bytes | |||
-rw-r--r-- | doc/contributing/doc_img/scatter-plot.png (renamed from doc/guides/doc_img/scatter-plot.png) | bin | 170802 -> 170802 bytes | |||
-rw-r--r-- | doc/contributing/internal-api.md (renamed from doc/guides/internal-api.md) | 0 | ||||
-rw-r--r-- | doc/contributing/investigating_native_memory_leak.md (renamed from doc/guides/investigating_native_memory_leak.md) | 0 | ||||
-rw-r--r-- | doc/contributing/issues.md (renamed from doc/guides/contributing/issues.md) | 0 | ||||
-rw-r--r-- | doc/contributing/maintaining-V8.md (renamed from doc/guides/maintaining-V8.md) | 0 | ||||
-rw-r--r-- | doc/contributing/maintaining-c-ares.md (renamed from doc/guides/maintaining-c-ares.md) | 2 | ||||
-rw-r--r-- | doc/contributing/maintaining-icu.md (renamed from doc/guides/maintaining-icu.md) | 0 | ||||
-rw-r--r-- | doc/contributing/maintaining-npm.md (renamed from doc/guides/maintaining-npm.md) | 0 | ||||
-rw-r--r-- | doc/contributing/maintaining-openssl.md (renamed from doc/guides/maintaining-openssl.md) | 0 | ||||
-rw-r--r-- | doc/contributing/maintaining-root-certs.md (renamed from doc/guides/maintaining-root-certs.md) | 0 | ||||
-rw-r--r-- | doc/contributing/maintaining-the-build-files.md (renamed from doc/guides/maintaining-the-build-files.md) | 0 | ||||
-rw-r--r-- | doc/contributing/maintaining-zlib.md (renamed from doc/guides/maintaining-zlib.md) | 2 | ||||
-rw-r--r-- | doc/contributing/node-postmortem-support.md (renamed from doc/guides/node-postmortem-support.md) | 0 | ||||
-rw-r--r-- | doc/contributing/offboarding.md (renamed from doc/guides/offboarding.md) | 0 | ||||
-rw-r--r-- | doc/contributing/pull-requests.md (renamed from doc/guides/contributing/pull-requests.md) | 16 | ||||
-rw-r--r-- | doc/contributing/releases.md (renamed from doc/guides/releases.md) | 2 | ||||
-rw-r--r-- | doc/contributing/security-release-process.md (renamed from doc/guides/security-release-process.md) | 0 | ||||
-rw-r--r-- | doc/contributing/security-steward-on-off-boarding.md (renamed from doc/guides/security-steward-on-off-boarding.md) | 0 | ||||
-rw-r--r-- | doc/contributing/static-analysis.md (renamed from doc/guides/static-analysis.md) | 0 | ||||
-rw-r--r-- | doc/contributing/strategic-initiatives.md (renamed from doc/guides/strategic-initiatives.md) | 0 | ||||
-rw-r--r-- | doc/contributing/technical-priorities.md (renamed from doc/guides/technical-priorities.md) | 0 | ||||
-rw-r--r-- | doc/contributing/technical-values.md (renamed from doc/guides/technical-values.md) | 0 | ||||
-rw-r--r-- | doc/contributing/using-internal-errors.md (renamed from doc/guides/using-internal-errors.md) | 0 | ||||
-rw-r--r-- | doc/contributing/using-symbols.md (renamed from doc/guides/using-symbols.md) | 0 | ||||
-rw-r--r-- | doc/contributing/writing-and-running-benchmarks.md (renamed from doc/guides/writing-and-running-benchmarks.md) | 0 | ||||
-rw-r--r-- | doc/contributing/writing-tests.md (renamed from doc/guides/writing-tests.md) | 0 | ||||
-rw-r--r-- | onboarding.md | 10 | ||||
-rw-r--r-- | src/README.md | 2 | ||||
-rw-r--r-- | src/base_object.h | 2 | ||||
-rw-r--r-- | src/env.h | 2 | ||||
-rw-r--r-- | src/handle_wrap.h | 2 | ||||
-rw-r--r-- | src/req_wrap.h | 2 | ||||
-rw-r--r-- | test/README.md | 4 | ||||
-rw-r--r-- | tools/icu/README.md | 4 |
53 files changed, 58 insertions, 60 deletions
diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 393964d59c5..4170038a0ac 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -13,9 +13,7 @@ /CODE_OF_CONDUCT.md @nodejs/tsc /CONTRIBUTING.md @nodejs/tsc /LICENSE @nodejs/tsc -/doc/guides/contributing/*.md @nodejs/tsc -/doc/guides/collaborator-guide.md @nodejs/tsc -/doc/guides/offboarding.md @nodejs/tsc +/doc/contributing/*.md @nodejs/tsc # streams @@ -88,7 +86,7 @@ /src/node_api* @nodejs/node-api /src/js_native_api* @nodejs/node-api -/doc/guides/adding-new-napi-api.md @nodejs/node-api +/doc/contributing/adding-new-napi-api.md @nodejs/node-api /doc/api/n-api.md @nodejs/node-api # gyp diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index f99a8abb394..fe46bf4dead 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -3,7 +3,7 @@ Before submitting a pull request, please read https://github.com/nodejs/node/blob/HEAD/CONTRIBUTING.md. Commit message formatting guidelines: -https://github.com/nodejs/node/blob/HEAD/doc/guides/contributing/pull-requests.md#commit-message-guidelines +https://github.com/nodejs/node/blob/HEAD/doc/contributing/pull-requests.md#commit-message-guidelines For code changes: 1. Include tests for any bug fixes or new features. diff --git a/.github/workflows/auto-start-ci.yml b/.github/workflows/auto-start-ci.yml index 6c392c5a5a6..632fdc0a970 100644 --- a/.github/workflows/auto-start-ci.yml +++ b/.github/workflows/auto-start-ci.yml @@ -5,7 +5,7 @@ on: # Runs every five minutes (fastest the scheduler can run). Five minutes is # optimistic, it can take longer to run. # To understand why `schedule` is used instead of other events, refer to - # ./doc/guides/commit-queue.md + # ./doc/contributing/commit-queue.md - cron: "*/5 * * * *" env: diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index d0d3f4f648e..f815adbb1fc 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -5,28 +5,28 @@ * [Pull Requests](#pull-requests) * [Developer's Certificate of Origin 1.1](#developers-certificate-of-origin) -## [Code of Conduct](./doc/guides/contributing/code-of-conduct.md) +## [Code of Conduct](./doc/contributing/code-of-conduct.md) The Node.js project has a [Code of Conduct](https://github.com/nodejs/admin/blob/HEAD/CODE_OF_CONDUCT.md) to which all contributors must adhere. -See [details on our policy on Code of Conduct](./doc/guides/contributing/code-of-conduct.md). +See [details on our policy on Code of Conduct](./doc/contributing/code-of-conduct.md). -## [Issues](./doc/guides/contributing/issues.md) +## [Issues](./doc/contributing/issues.md) -* [Asking for General Help](./doc/guides/contributing/issues.md#asking-for-general-help) -* [Discussing non-technical topics](./doc/guides/contributing/issues.md#discussing-non-technical-topics) -* [Submitting a Bug Report](./doc/guides/contributing/issues.md#submitting-a-bug-report) -* [Triaging a Bug Report](./doc/guides/contributing/issues.md#triaging-a-bug-report) +* [Asking for General Help](./doc/contributing/issues.md#asking-for-general-help) +* [Discussing non-technical topics](./doc/contributing/issues.md#discussing-non-technical-topics) +* [Submitting a Bug Report](./doc/contributing/issues.md#submitting-a-bug-report) +* [Triaging a Bug Report](./doc/contributing/issues.md#triaging-a-bug-report) -## [Pull Requests](./doc/guides/contributing/pull-requests.md) +## [Pull Requests](./doc/contributing/pull-requests.md) -* [Dependencies](./doc/guides/contributing/pull-requests.md#dependencies) -* [Setting up your local environment](./doc/guides/contributing/pull-requests.md#setting-up-your-local-environment) -* [The Process of Making Changes](./doc/guides/contributing/pull-requests.md#the-process-of-making-changes) -* [Reviewing Pull Requests](./doc/guides/contributing/pull-requests.md#reviewing-pull-requests) -* [Notes](./doc/guides/contributing/pull-requests.md#notes) +* [Dependencies](./doc/contributing/pull-requests.md#dependencies) +* [Setting up your local environment](./doc/contributing/pull-requests.md#setting-up-your-local-environment) +* [The Process of Making Changes](./doc/contributing/pull-requests.md#the-process-of-making-changes) +* [Reviewing Pull Requests](./doc/contributing/pull-requests.md#reviewing-pull-requests) +* [Notes](./doc/contributing/pull-requests.md#notes) <a id="developers-certificate-of-origin"></a> diff --git a/GOVERNANCE.md b/GOVERNANCE.md index fb74525eb0e..f5f3ffa015c 100644 --- a/GOVERNANCE.md +++ b/GOVERNANCE.md @@ -25,7 +25,7 @@ Triagers are given the "Triage" GitHub role and have: See: * [List of triagers](./README.md#triagers) -* [A guide for triagers](./doc/guides/contributing/issues.md#triaging-a-bug-report) +* [A guide for triagers](./doc/contributing/issues.md#triaging-a-bug-report) ## Collaborators @@ -54,7 +54,7 @@ result in collaborators removing their opposition. See: * [List of collaborators](./README.md#current-project-team-members) -* [A guide for collaborators](./doc/guides/collaborator-guide.md) +* [A guide for collaborators](./doc/contributing/collaborator-guide.md) ### Collaborator activities @@ -997,14 +997,14 @@ endif .PHONY: release-only release-only: check-xz @if [ "$(DISTTYPE)" = "release" ] && `grep -q REPLACEME doc/api/*.md`; then \ - echo 'Please update REPLACEME tags in the following doc/api/*.md files (See doc/guides/releases.md):\n' ; \ + echo 'Please update REPLACEME tags in the following doc/api/*.md files (See doc/contributing/releases.md):\n' ; \ REPLACEMES="$(shell grep -l REPLACEME doc/api/*.md)" ; \ echo "$$REPLACEMES\n" | tr " " "\n" ; \ exit 1 ; \ fi @if [ "$(DISTTYPE)" = "release" ] && \ `grep -q DEP...X doc/api/deprecations.md`; then \ - echo 'Please update DEP...X in doc/api/deprecations.md (See doc/guides/releases.md)' ; \ + echo 'Please update DEP...X in doc/api/deprecations.md (See doc/contributing/releases.md)' ; \ exit 1 ; \ fi @if [ "$(shell git status --porcelain | egrep -v '^\?\? ')" = "" ]; then \ diff --git a/README.md b/README.md index 53de55288a0..93f803420ee 100644 --- a/README.md +++ b/README.md @@ -635,7 +635,7 @@ For information about the governance of the Node.js project, see <!--lint enable prohibited-strings--> -Collaborators follow the [Collaborator Guide](./doc/guides/collaborator-guide.md) in +Collaborators follow the [Collaborator Guide](./doc/contributing/collaborator-guide.md) in maintaining the Node.js project. ### Triagers @@ -741,6 +741,6 @@ license text. [Contributing to the project]: CONTRIBUTING.md [Node.js Website]: https://nodejs.org/ [OpenJS Foundation]: https://openjsf.org/ -[Strategic initiatives]: doc/guides/strategic-initiatives.md -[Technical values and prioritization]: doc/guides/technical-values.md +[Strategic initiatives]: doc/contributing/strategic-initiatives.md +[Technical values and prioritization]: doc/contributing/technical-values.md [Working Groups]: https://github.com/nodejs/TSC/blob/HEAD/WORKING_GROUPS.md diff --git a/benchmark/README.md b/benchmark/README.md index e40972fde08..710731b3c9f 100644 --- a/benchmark/README.md +++ b/benchmark/README.md @@ -5,7 +5,7 @@ of different Node.js implementations and different ways of writing JavaScript run by the built-in JavaScript engine. For a detailed guide on how to write and run benchmarks in this -directory, see [the guide on benchmarks](../doc/guides/writing-and-running-benchmarks.md). +directory, see [the guide on benchmarks](../doc/contributing/writing-and-running-benchmarks.md). ## Table of Contents @@ -77,17 +77,17 @@ writing benchmarks. ### `createBenchmark(fn, configs[, options])` -See [the guide on writing benchmarks](../doc/guides/writing-and-running-benchmarks.md#basics-of-a-benchmark). +See [the guide on writing benchmarks](../doc/contributing/writing-and-running-benchmarks.md#basics-of-a-benchmark). ### `default_http_benchmarker` The default benchmarker used to run HTTP benchmarks. -See [the guide on writing HTTP benchmarks](../doc/guides/writing-and-running-benchmarks.md#creating-an-http-benchmark). +See [the guide on writing HTTP benchmarks](../doc/contributing/writing-and-running-benchmarks.md#creating-an-http-benchmark). ### `PORT` The default port used to run HTTP benchmarks. -See [the guide on writing HTTP benchmarks](../doc/guides/writing-and-running-benchmarks.md#creating-an-http-benchmark). +See [the guide on writing HTTP benchmarks](../doc/contributing/writing-and-running-benchmarks.md#creating-an-http-benchmark). ### `sendResult(data)` diff --git a/deps/openssl/README.md b/deps/openssl/README.md index bd0ad0c9949..225bfba8e74 100644 --- a/deps/openssl/README.md +++ b/deps/openssl/README.md @@ -77,4 +77,4 @@ Please refer [config/opensslconf_asm.h](config/opensslconf_asm.h) for details. ### Upgrading OpenSSL -Please refer to [maintaining-openssl](../../doc/guides/maintaining-openssl.md). +Please refer to [maintaining-openssl](../../doc/contributing/maintaining-openssl.md). diff --git a/doc/guides/adding-new-napi-api.md b/doc/contributing/adding-new-napi-api.md index 18f551929e4..18f551929e4 100644 --- a/doc/guides/adding-new-napi-api.md +++ b/doc/contributing/adding-new-napi-api.md diff --git a/doc/guides/backporting-to-release-lines.md b/doc/contributing/backporting-to-release-lines.md index 389c979e2ee..389c979e2ee 100644 --- a/doc/guides/backporting-to-release-lines.md +++ b/doc/contributing/backporting-to-release-lines.md diff --git a/doc/guides/building-node-with-ninja.md b/doc/contributing/building-node-with-ninja.md index 5231b73a8cd..5231b73a8cd 100644 --- a/doc/guides/building-node-with-ninja.md +++ b/doc/contributing/building-node-with-ninja.md diff --git a/doc/guides/contributing/code-of-conduct.md b/doc/contributing/code-of-conduct.md index 1332246e404..1332246e404 100644 --- a/doc/guides/contributing/code-of-conduct.md +++ b/doc/contributing/code-of-conduct.md diff --git a/doc/guides/collaborator-guide.md b/doc/contributing/collaborator-guide.md index 9911acdd973..77e88f4a457 100644 --- a/doc/guides/collaborator-guide.md +++ b/doc/contributing/collaborator-guide.md @@ -408,7 +408,7 @@ For pull requests introducing new core modules: Node-API provides an ABI-stable API guaranteed for future Node.js versions. Node-API additions call for unusual care and scrutiny. If a change adds to `node_api.h`, `js_native_api.h`, `node_api_types.h`, or `js_native_api_types.h`, -consult [the relevant guide](https://github.com/nodejs/node/blob/HEAD/doc/guides/adding-new-napi-api.md). +consult [the relevant guide](https://github.com/nodejs/node/blob/HEAD/doc/contributing/adding-new-napi-api.md). ### Deprecations @@ -558,7 +558,7 @@ $ git checkout master ``` Update the tree (assumes your repository is set up as detailed in -[CONTRIBUTING.md](./contributing/pull-requests.md#step-1-fork)): +[CONTRIBUTING.md](./pull-requests.md#step-1-fork)): ```text $ git fetch upstream @@ -925,7 +925,7 @@ need to be attached anymore, as only important bugfixes will be included. [`--throw-deprecation`]: ../api/cli.md#--throw-deprecation [`node-core-utils`]: https://github.com/nodejs/node-core-utils [backporting guide]: backporting-to-release-lines.md -[commit message guidelines]: contributing/pull-requests.md#commit-message-guidelines +[commit message guidelines]: pull-requests.md#commit-message-guidelines [commit-example]: https://github.com/nodejs/node/commit/b636ba8186 [commit-queue.md]: ./commit-queue.md [git-email]: https://help.github.com/articles/setting-your-commit-email-address-in-git/ diff --git a/doc/guides/commit-queue.md b/doc/contributing/commit-queue.md index edb9c7ad4a8..edb9c7ad4a8 100644 --- a/doc/guides/commit-queue.md +++ b/doc/contributing/commit-queue.md diff --git a/doc/guides/components-in-core.md b/doc/contributing/components-in-core.md index 2cae1c40e59..2cae1c40e59 100644 --- a/doc/guides/components-in-core.md +++ b/doc/contributing/components-in-core.md diff --git a/doc/guides/cpp-style-guide.md b/doc/contributing/cpp-style-guide.md index 0de0d2ec5ba..64d0efee911 100644 --- a/doc/guides/cpp-style-guide.md +++ b/doc/contributing/cpp-style-guide.md @@ -403,5 +403,5 @@ even `try` and `catch` **will** break. [Run Time Type Information]: https://en.wikipedia.org/wiki/Run-time_type_information [aliased_buffer.h]: https://github.com/nodejs/node/blob/HEAD/src/aliased_buffer.h#L12 [cppref_auto_ptr]: https://en.cppreference.com/w/cpp/memory/auto_ptr -[errors]: https://github.com/nodejs/node/blob/HEAD/doc/guides/using-internal-errors.md +[errors]: https://github.com/nodejs/node/blob/HEAD/doc/contributing/using-internal-errors.md [without C++ exception handling]: https://gcc.gnu.org/onlinedocs/libstdc++/manual/using_exceptions.html#intro.using.exception.no diff --git a/doc/guides/diagnostic-tooling-support-tiers.md b/doc/contributing/diagnostic-tooling-support-tiers.md index 556c87e6a95..556c87e6a95 100644 --- a/doc/guides/diagnostic-tooling-support-tiers.md +++ b/doc/contributing/diagnostic-tooling-support-tiers.md diff --git a/doc/guides/doc_img/compare-boxplot.png b/doc/contributing/doc_img/compare-boxplot.png Binary files differindex 2a198a44913..2a198a44913 100644 --- a/doc/guides/doc_img/compare-boxplot.png +++ b/doc/contributing/doc_img/compare-boxplot.png diff --git a/doc/guides/doc_img/scatter-plot.png b/doc/contributing/doc_img/scatter-plot.png Binary files differindex 726129e6334..726129e6334 100644 --- a/doc/guides/doc_img/scatter-plot.png +++ b/doc/contributing/doc_img/scatter-plot.png diff --git a/doc/guides/internal-api.md b/doc/contributing/internal-api.md index 2631978e1af..2631978e1af 100644 --- a/doc/guides/internal-api.md +++ b/doc/contributing/internal-api.md diff --git a/doc/guides/investigating_native_memory_leak.md b/doc/contributing/investigating_native_memory_leak.md index 8c193d1285f..8c193d1285f 100644 --- a/doc/guides/investigating_native_memory_leak.md +++ b/doc/contributing/investigating_native_memory_leak.md diff --git a/doc/guides/contributing/issues.md b/doc/contributing/issues.md index 7cd043148ff..7cd043148ff 100644 --- a/doc/guides/contributing/issues.md +++ b/doc/contributing/issues.md diff --git a/doc/guides/maintaining-V8.md b/doc/contributing/maintaining-V8.md index 339cda49488..339cda49488 100644 --- a/doc/guides/maintaining-V8.md +++ b/doc/contributing/maintaining-V8.md diff --git a/doc/guides/maintaining-c-ares.md b/doc/contributing/maintaining-c-ares.md index 968b96f2582..64f29674c4e 100644 --- a/doc/guides/maintaining-c-ares.md +++ b/doc/contributing/maintaining-c-ares.md @@ -62,5 +62,5 @@ Commit the changes with a message like ```text deps: update c-ares to x.y.z -Updated as described in doc/guides/maintaining-c-ares.md. +Updated as described in doc/contributing/maintaining-c-ares.md. ``` diff --git a/doc/guides/maintaining-icu.md b/doc/contributing/maintaining-icu.md index 73801f60ffe..73801f60ffe 100644 --- a/doc/guides/maintaining-icu.md +++ b/doc/contributing/maintaining-icu.md diff --git a/doc/guides/maintaining-npm.md b/doc/contributing/maintaining-npm.md index 5536fd513f8..5536fd513f8 100644 --- a/doc/guides/maintaining-npm.md +++ b/doc/contributing/maintaining-npm.md diff --git a/doc/guides/maintaining-openssl.md b/doc/contributing/maintaining-openssl.md index 25a58e2f440..25a58e2f440 100644 --- a/doc/guides/maintaining-openssl.md +++ b/doc/contributing/maintaining-openssl.md diff --git a/doc/guides/maintaining-root-certs.md b/doc/contributing/maintaining-root-certs.md index d7fb05a4f24..d7fb05a4f24 100644 --- a/doc/guides/maintaining-root-certs.md +++ b/doc/contributing/maintaining-root-certs.md diff --git a/doc/guides/maintaining-the-build-files.md b/doc/contributing/maintaining-the-build-files.md index 67c86281c63..67c86281c63 100644 --- a/doc/guides/maintaining-the-build-files.md +++ b/doc/contributing/maintaining-the-build-files.md diff --git a/doc/guides/maintaining-zlib.md b/doc/contributing/maintaining-zlib.md index 4dba29bacf9..caed2534c69 100644 --- a/doc/guides/maintaining-zlib.md +++ b/doc/contributing/maintaining-zlib.md @@ -32,5 +32,5 @@ Commit the changes with a message like ```text deps: update zlib to upstream d7f3ca9 -Updated as described in doc/guides/maintaining-zlib.md. +Updated as described in doc/contributing/maintaining-zlib.md. ``` diff --git a/doc/guides/node-postmortem-support.md b/doc/contributing/node-postmortem-support.md index 2136f953dcf..2136f953dcf 100644 --- a/doc/guides/node-postmortem-support.md +++ b/doc/contributing/node-postmortem-support.md diff --git a/doc/guides/offboarding.md b/doc/contributing/offboarding.md index 87f21103540..87f21103540 100644 --- a/doc/guides/offboarding.md +++ b/doc/contributing/offboarding.md diff --git a/doc/guides/contributing/pull-requests.md b/doc/contributing/pull-requests.md index 95024008645..00868c4bfc9 100644 --- a/doc/guides/contributing/pull-requests.md +++ b/doc/contributing/pull-requests.md @@ -137,8 +137,8 @@ added: REPLACEME ``` For contributing C++ code, you may want to look at the -[C++ Style Guide](../cpp-style-guide.md), as well as the -[README of `src/`](../../../src/README.md) for an overview of Node.js +[C++ Style Guide](cpp-style-guide.md), as well as the +[README of `src/`](../../src/README.md) for an overview of Node.js C++ internals. ### Step 4: Commit @@ -586,16 +586,16 @@ You can find the full list of supported subsystems in the [nodejs/core-validate-commit][] repository. More than one subsystem may be valid for any particular issue or pull request. -[Building guide]: ../../../BUILDING.md +[Building guide]: ../../BUILDING.md [CI (Continuous Integration) test run]: #ci-testing [Code of Conduct]: https://github.com/nodejs/admin/blob/HEAD/CODE_OF_CONDUCT.md -[Onboarding guide]: ../../../onboarding.md +[Onboarding guide]: ../../onboarding.md [approved]: #getting-approvals-for-your-pull-request -[benchmark results]: ../writing-and-running-benchmarks.md -[collaborator guide]: ../collaborator-guide.md -[guide for writing tests in Node.js]: ../writing-tests.md +[benchmark results]: writing-and-running-benchmarks.md +[collaborator guide]: collaborator-guide.md +[guide for writing tests in Node.js]: writing-tests.md [hiding-a-comment]: https://help.github.com/articles/managing-disruptive-comments/#hiding-a-comment [https://ci.nodejs.org/]: https://ci.nodejs.org/ [nodejs/core-validate-commit]: https://github.com/nodejs/core-validate-commit/blob/main/lib/rules/subsystem.js [pull request template]: https://raw.githubusercontent.com/nodejs/node/HEAD/.github/PULL_REQUEST_TEMPLATE.md -[running tests]: ../../../BUILDING.md#running-tests +[running tests]: ../../BUILDING.md#running-tests diff --git a/doc/guides/releases.md b/doc/contributing/releases.md index a3a3b51f8bf..042f8283744 100644 --- a/doc/guides/releases.md +++ b/doc/contributing/releases.md @@ -202,7 +202,7 @@ When cherry-picking commits, if there are simple conflicts you can resolve them. Otherwise, add the `backport-requested-vN.x` label to the original PR and post a comment stating that it does not land cleanly and will require a backport PR. You can refer the owner of the PR to the "[Backporting to Release -Lines](https://github.com/nodejs/node/blob/HEAD/doc/guides/backporting-to-release-lines.md)" guide. +Lines](https://github.com/nodejs/node/blob/HEAD/doc/contributing/backporting-to-release-lines.md)" guide. If commits were cherry-picked in this step, check that the test still pass. diff --git a/doc/guides/security-release-process.md b/doc/contributing/security-release-process.md index 7fbf68152dd..7fbf68152dd 100644 --- a/doc/guides/security-release-process.md +++ b/doc/contributing/security-release-process.md diff --git a/doc/guides/security-steward-on-off-boarding.md b/doc/contributing/security-steward-on-off-boarding.md index 19c058f1696..19c058f1696 100644 --- a/doc/guides/security-steward-on-off-boarding.md +++ b/doc/contributing/security-steward-on-off-boarding.md diff --git a/doc/guides/static-analysis.md b/doc/contributing/static-analysis.md index 6130f653580..6130f653580 100644 --- a/doc/guides/static-analysis.md +++ b/doc/contributing/static-analysis.md diff --git a/doc/guides/strategic-initiatives.md b/doc/contributing/strategic-initiatives.md index 8bcd8800b7e..8bcd8800b7e 100644 --- a/doc/guides/strategic-initiatives.md +++ b/doc/contributing/strategic-initiatives.md diff --git a/doc/guides/technical-priorities.md b/doc/contributing/technical-priorities.md index 924572b206d..924572b206d 100644 --- a/doc/guides/technical-priorities.md +++ b/doc/contributing/technical-priorities.md diff --git a/doc/guides/technical-values.md b/doc/contributing/technical-values.md index f6d41207d16..f6d41207d16 100644 --- a/doc/guides/technical-values.md +++ b/doc/contributing/technical-values.md diff --git a/doc/guides/using-internal-errors.md b/doc/contributing/using-internal-errors.md index b55b187bff0..b55b187bff0 100644 --- a/doc/guides/using-internal-errors.md +++ b/doc/contributing/using-internal-errors.md diff --git a/doc/guides/using-symbols.md b/doc/contributing/using-symbols.md index 2bc32862b71..2bc32862b71 100644 --- a/doc/guides/using-symbols.md +++ b/doc/contributing/using-symbols.md diff --git a/doc/guides/writing-and-running-benchmarks.md b/doc/contributing/writing-and-running-benchmarks.md index a5c52eafb8a..a5c52eafb8a 100644 --- a/doc/guides/writing-and-running-benchmarks.md +++ b/doc/contributing/writing-and-running-benchmarks.md diff --git a/doc/guides/writing-tests.md b/doc/contributing/writing-tests.md index 9884408af85..9884408af85 100644 --- a/doc/guides/writing-tests.md +++ b/doc/contributing/writing-tests.md diff --git a/onboarding.md b/onboarding.md index eef5ee41e70..f17b24c50f1 100644 --- a/onboarding.md +++ b/onboarding.md @@ -253,15 +253,15 @@ needs to be pointed out separately during the onboarding. access to the projects coverity project as outlined in [static-analysis][]. [Code of Conduct]: https://github.com/nodejs/admin/blob/HEAD/CODE_OF_CONDUCT.md -[Labels]: doc/guides/collaborator-guide.md#labels -[Landing pull requests]: doc/guides/collaborator-guide.md#landing-pull-requests +[Labels]: doc/contributing/collaborator-guide.md#labels +[Landing pull requests]: doc/contributing/collaborator-guide.md#landing-pull-requests [Publicizing or hiding organization membership]: https://help.github.com/articles/publicizing-or-hiding-organization-membership/ -[`author-ready`]: doc/guides/collaborator-guide.md#author-ready-pull-requests +[`author-ready`]: doc/contributing/collaborator-guide.md#author-ready-pull-requests [`core-validate-commit`]: https://github.com/nodejs/core-validate-commit [`git-node`]: https://github.com/nodejs/node-core-utils/blob/HEAD/docs/git-node.md [`node-core-utils`]: https://github.com/nodejs/node-core-utils [set up the credentials]: https://github.com/nodejs/node-core-utils#setting-up-github-credentials -[static-analysis]: doc/guides/static-analysis.md +[static-analysis]: doc/contributing/static-analysis.md [two-factor authentication]: https://help.github.com/articles/securing-your-account-with-two-factor-authentication-2fa/ [using a TOTP mobile app]: https://help.github.com/articles/configuring-two-factor-authentication-via-a-totp-mobile-app/ -[who-to-cc]: doc/guides/collaborator-guide.md#who-to-cc-in-the-issue-tracker +[who-to-cc]: doc/contributing/collaborator-guide.md#who-to-cc-in-the-issue-tracker diff --git a/src/README.md b/src/README.md index 25c63e2f8cc..623c3774971 100644 --- a/src/README.md +++ b/src/README.md @@ -1024,7 +1024,7 @@ static void GetUserInfo(const FunctionCallbackInfo<Value>& args) { } ``` -[C++ coding style]: ../doc/guides/cpp-style-guide.md +[C++ coding style]: ../doc/contributing/cpp-style-guide.md [Callback scopes]: #callback-scopes [JavaScript value handles]: #js-handles [N-API]: https://nodejs.org/api/n-api.html diff --git a/src/base_object.h b/src/base_object.h index d46a0f21600..1c63da92fd8 100644 --- a/src/base_object.h +++ b/src/base_object.h @@ -171,7 +171,7 @@ class BaseObject : public MemoryRetainer { // class because it is used by src/node_postmortem_metadata.cc to calculate // offsets and generate debug symbols for BaseObject, which assumes that the // position of members in memory are predictable. For more information please - // refer to `doc/guides/node-postmortem-support.md` + // refer to `doc/contributing/node-postmortem-support.md` friend int GenDebugSymbols(); friend class CleanupHookCallback; template <typename T, bool kIsWeak> diff --git a/src/env.h b/src/env.h index dfb5155b9b8..86891290416 100644 --- a/src/env.h +++ b/src/env.h @@ -1561,7 +1561,7 @@ class Environment : public MemoryRetainer { // src/node_postmortem_metadata.cc to calculate offsets and generate debug // symbols for Environment, which assumes that the position of members in // memory are predictable. For more information please refer to - // `doc/guides/node-postmortem-support.md` + // `doc/contributing/node-postmortem-support.md` friend int GenDebugSymbols(); HandleWrapQueue handle_wrap_queue_; ReqWrapQueue req_wrap_queue_; diff --git a/src/handle_wrap.h b/src/handle_wrap.h index ff25db2d919..2e06829b7bd 100644 --- a/src/handle_wrap.h +++ b/src/handle_wrap.h @@ -106,7 +106,7 @@ class HandleWrap : public AsyncWrap { // class because it is used by src/node_postmortem_metadata.cc to calculate // offsets and generate debug symbols for HandleWrap, which assumes that the // position of members in memory are predictable. For more information please - // refer to `doc/guides/node-postmortem-support.md` + // refer to `doc/contributing/node-postmortem-support.md` friend int GenDebugSymbols(); ListNode<HandleWrap> handle_wrap_queue_; enum { kInitialized, kClosing, kClosed } state_; diff --git a/src/req_wrap.h b/src/req_wrap.h index 5d7fcb42d01..611e438275a 100644 --- a/src/req_wrap.h +++ b/src/req_wrap.h @@ -66,7 +66,7 @@ class ReqWrap : public AsyncWrap, public ReqWrapBase { // members in memory are predictable. sizeof(req_) depends on the type of T, // so req_wrap_queue_ would no longer be at a fixed offset if it came after // req_. For more information please refer to - // `doc/guides/node-postmortem-support.md` + // `doc/contributing/node-postmortem-support.md` T req_; }; diff --git a/test/README.md b/test/README.md index 0534d0dfb81..121ecc89b54 100644 --- a/test/README.md +++ b/test/README.md @@ -3,10 +3,10 @@ This directory contains code and data used to test the Node.js implementation. For a detailed guide on how to write tests in this -directory, see [the guide on writing tests](../doc/guides/writing-tests.md). +directory, see [the guide on writing tests](../doc/contributing/writing-tests.md). On how to run tests in this directory, see -[the contributing guide](../doc/guides/contributing/pull-requests.md#step-6-test). +[the contributing guide](../doc/contributing/pull-requests.md#step-6-test). For the tests to run on Windows, be sure to clone Node.js source code with the `autocrlf` git config flag set to true. diff --git a/tools/icu/README.md b/tools/icu/README.md index 544f4bf2152..2e908e2cf44 100644 --- a/tools/icu/README.md +++ b/tools/icu/README.md @@ -27,8 +27,8 @@ Note: ## See Also -* [docs/guides/maintaining-icu.md](../../doc/guides/maintaining-icu.md) for - information on maintaining ICU in Node.js +* [docs/guides/maintaining-icu.md](../../doc/contributing/maintaining-icu.md) + for information on maintaining ICU in Node.js * [docs/api/intl.md](../../doc/api/intl.md) for information on the internationalization-related APIs in Node.js |