doc: update exercise portion of onboarding doc

PR-URL: #8559 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Ilkka Myller <[email protected]>
nodejs · Oct 10, 2016 · 48d64fc · 48d64fc
1 parent c0f79da
commit 48d64fc
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/doc/onboarding.md b/doc/onboarding.md
@@ -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