Skip to content

Commit

Permalink
doc: update exercise portion of onboarding doc
Browse files Browse the repository at this point in the history
PR-URL: #8559
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Ilkka Myller <ilkka.myller@nodefield.com>
  • Loading branch information
Trott authored and MylesBorins committed Oct 26, 2016
1 parent 5a07bb6 commit d123fc1
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions doc/onboarding.md
Original file line number Diff line number Diff line change
Expand Up @@ -184,14 +184,15 @@ Landing a PR
* close the original PR with "Landed in `<commit hash>`".


## exercise: make PRs adding yourselves to the README
## Exercise: Make a PR adding yourself to the README

* Example: https://github.com/nodejs/node/commit/7b09aade8468e1c930f36b9c81e6ac2ed5bc8732
* to see full URL: `git log 7b09aade8468e1c930f36b9c81e6ac2ed5bc8732 -1`
* Collaborators in alphabetical order by username
* Label your pull request with the `doc` subsystem label
* If you would like to run CI on your PR, feel free to
* Make sure to added the `PR-URL: <full-pr-url>`!
* For raw commit message: `git log 7b09aade8468e1c930f36b9c81e6ac2ed5bc8732 -1`
* Collaborators are in alphabetical order by GitHub username.
* Label your pull request with the `doc` subsystem label.
* Run CI on your PR.
* After a `LGTM` or two, land the PR.
* Be sure to add the `PR-URL: <full-pr-url>` and appropriate `Reviewed-By:` metadata!


## final notes
Expand Down

0 comments on commit d123fc1

Please sign in to comment.