From 6513a589fe5e8da18a6b8819fbffe558081d57df Mon Sep 17 00:00:00 2001 From: "Pooja D.P" Date: Tue, 20 Oct 2020 16:13:21 +0400 Subject: [PATCH] doc: add a subsystems header in pull-requests.md PR-URL: https://github.com/nodejs/node/pull/35718 Reviewed-By: Rich Trott Reviewed-By: James M Snell Reviewed-By: Gireesh Punathil --- doc/guides/contributing/pull-requests.md | 18 +++++++++++++++--- doc/guides/onboarding-extras.md | 11 ----------- 2 files changed, 15 insertions(+), 14 deletions(-) diff --git a/doc/guides/contributing/pull-requests.md b/doc/guides/contributing/pull-requests.md index 84039f89412b71..c61ac5ff37d0a1 100644 --- a/doc/guides/contributing/pull-requests.md +++ b/doc/guides/contributing/pull-requests.md @@ -35,6 +35,7 @@ so that you can make the actual changes. This is where we will start. * [CI Testing](#ci-testing) * [Waiting Until the Pull Request Gets Landed](#waiting-until-the-pull-request-gets-landed) * [Check Out the Collaborator Guide](#check-out-the-collaborator-guide) + * [Appendix: Subsystems](#appendix-subsystems) ## Dependencies @@ -171,9 +172,9 @@ A good commit message should describe what changed and why. less, and no more than 72 characters) * be entirely in lowercase with the exception of proper nouns, acronyms, and the words that refer to code, like function/variable names - * be prefixed with the name of the changed subsystem and start with an - imperative verb. Check the output of `git log --oneline files/you/changed` to - find out what subsystems your changes touch. + * be prefixed with the name of the changed [subsystem](#appendix-subsystems) + and start with an imperative verb. Check the output of `git log --oneline + files/you/changed` to find out what subsystems your changes touch. Examples: * `net: add localAddress and localPort to Socket` @@ -603,6 +604,17 @@ widely used, so don't be discouraged! If you want to know more about the code review and the landing process, see the [Collaborator Guide][]. +### Appendix: Subsystems + +* `lib/*.js` (`assert`, `buffer`, etc.) +* `build` +* `doc` +* `lib / src` +* `test` +* `tools` + +More than one subsystem may be valid for any particular issue or pull request. + [Building guide]: ../../../BUILDING.md [CI (Continuous Integration) test run]: #ci-testing [Code of Conduct]: https://github.com/nodejs/admin/blob/master/CODE_OF_CONDUCT.md diff --git a/doc/guides/onboarding-extras.md b/doc/guides/onboarding-extras.md index 030074093eef36..79951a433926c6 100644 --- a/doc/guides/onboarding-extras.md +++ b/doc/guides/onboarding-extras.md @@ -2,17 +2,6 @@ ## Labels -### Subsystems - -* `lib/*.js` (`assert`, `buffer`, etc.) -* `build` -* `doc` -* `lib / src` -* `test` -* `tools` - -More than one subsystem may be valid for any particular issue or pull request. - ### General * `confirmed-bug`: Bugs you have verified