doc: simplify COLLABORATOR_GUIDE.md instructions

Remove bulleted list formatting. Only check email/name for new
contributors. Change `full name` to `name` and remove italics to avoid
incorrect implication that we need a "legal name".

PR-URL: https://github.com/nodejs/node/pull/19458
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
This commit is contained in:
Rich Trott 2018-03-19 12:04:29 -07:00
Родитель f3e078fdce
Коммит a72bfb94dd
1 изменённых файлов: 4 добавлений и 5 удалений

Просмотреть файл

@ -494,12 +494,11 @@ The TSC should serve as the final arbiter where required.
one](https://github.com/nodejs/node/commit/b636ba8186) if unsure exactly how
to format your commit messages.
Additionally:
Check PRs from new contributors to make sure the person's name and email address
are correct before merging.
* Double check PRs to make sure the person's _full name_ and email
address are correct before merging.
* All commits should be self-contained (meaning every commit should pass all
tests). This makes it much easier when bisecting to find a breaking change.
All commits should be self-contained, meaning every commit should pass all
tests. This makes it much easier when bisecting to find a breaking change.
### Using `git-node`