Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: improve onboarding extras formatting #6548

Merged
merged 1 commit into from
May 9, 2016
Merged

doc: improve onboarding extras formatting #6548

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
44 changes: 22 additions & 22 deletions doc/onboarding-extras.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,26 +1,24 @@
## Who to CC in issues

* `lib/buffer`: @trevnorris
* `lib/child_process`: @cjihrig, @bnoordhuis, @piscisaereus
* `lib/cluster`: @cjihrig, @bnoordhuis, @piscisaereus
* `lib/{crypto,tls,https}`: @indutny, @shigeki, @nodejs/crypto
* `lib/domains`: @misterdjules
* `lib/{_}http{*}`: @indutny, @bnoordhuis, @nodejs/http
* `lib/net`: @indutny, @bnoordhuis, @piscisaereus, @chrisdickinson, @nodejs/streams
* `lib/{_}stream{s|*}`: @nodejs/streams
* `lib/repl`: @fishrock123
* `lib/timers`: @fishrock123, @misterdjules
* `lib/zlib`: @indutny, @bnoordhuis

* `src/async-wrap.*`: @trevnorris
* `src/node_crypto.*`: @indutny, @shigeki, @nodejs/crypto

* `test/*`: @nodejs/testing, @trott

* `tools/eslint`, `.eslintrc`: @silverwind, @trott

* upgrading v8: @bnoordhuis / @targos / @ofrobots
* upgrading npm: @thealphanerd, @fishrock123
| subsystem | maintainers |
| --- | --- |
| `lib/buffer` | @trevnorris |
| `lib/child_process` | @cjihrig, @bnoordhuis |
| `lib/cluster` | @cjihrig, @bnoordhuis |
| `lib/{crypto,tls,https}` | @nodejs/crypto |
| `lib/domains` | @misterdjules |
| `lib/{_}http{*}` | @indutny, @bnoordhuis, @mscdex, @nodejs/http |
| `lib/net` | @indutny, @bnoordhuis, @nodejs/streams |
| `lib/{_}stream{s|*}` | @nodejs/streams |
| `lib/repl` | @fishrock123 |
| `lib/timers` | @fishrock123, @misterdjules |
| `lib/zlib` | @indutny, @bnoordhuis |
| `src/async-wrap.*` | @trevnorris |
| `src/node_crypto.*` | @nodejs/crypto |
| `test/*` | @nodejs/testing |
| `tools/eslint`, `.eslintrc` | @silverwind, @trott |
| upgrading v8 | @bnoordhuis, @targos, @ofrobots |
| upgrading npm | @thealphanerd, @fishrock123 |


When things need extra attention, are controversial, or `semver-major`: @nodejs/ctc
Expand Down Expand Up @@ -52,6 +50,8 @@ Please use these when possible / appropriate
* `feature request` - Any issue that requests a new feature (usually not PRs)
* `good first contribution` - Issues suitable for newcomers to process

--

* `semver-{minor,major}`
* be conservative – that is, if a change has the remote *chance* of breaking something, go for semver-major
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

linewrap

Copy link
Contributor Author

@Fishrock123 Fishrock123 May 3, 2016 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Splitting doesn't work for markdown lists with some renderers, sorry. :(

* when adding a semver label, add a comment explaining why you're adding it
Expand Down Expand Up @@ -94,7 +94,7 @@ to update from nodejs/node:
* prefer to make the originating user update the code, since they have it fresh in mind
* first, reattempt with `git am -3` (3-way merge)`
* if `-3` still fails, and you need to get it merged:
* `git fetch origin pull/N/head:pr-N && git checkout pr-N && git rebase master`
* `git fetch upstream pull/N/head:pr-N && git checkout pr-N && git rebase master`


## best practices
Expand Down