cordova-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject cordova-coho git commit: Update to no longer mention reviewboard
Date Fri, 09 Jan 2015 02:38:24 GMT
Repository: cordova-coho
Updated Branches:
  refs/heads/master 7a568a09e -> 03779576d

Update to no longer mention reviewboard


Branch: refs/heads/master
Commit: 03779576da154c1ec4c0df18d07476ab0cfd331d
Parents: 7a568a0
Author: Andrew Grieve <>
Authored: Thu Jan 8 21:37:57 2015 -0500
Committer: Andrew Grieve <>
Committed: Thu Jan 8 21:37:57 2015 -0500

 docs/ | 23 ++++-------------------
 1 file changed, 4 insertions(+), 19 deletions(-)
diff --git a/docs/ b/docs/
index 3136f70..8b4e303 100644
--- a/docs/
+++ b/docs/
@@ -21,23 +21,8 @@
 # Code Reviews
-TODO: Review Board didn't really work out :(. Come up with something better.
+We use GitHub pull requests for code reviews.
+1. Commit to a branch on your own fork of the repo that you're changing
+2. Use GitHub to create a pull request
+3. Email out the link to it
- * Use to create a review request.
-   * You will need a Review Board userid and password. One can be requested from the site.
-   * By default, the ML will be notified of your review request.
-   * If you have someone in particular that you would like approval from, be sure to add
them in "People" field of the review.
-   * To create your review request is to use [post-review](
(RBTools) from the repo with the change, only if the repo contains the file '.reviewboardrc'
-     Currently the following repos contain .reviewboardrc:
-     cordova-coho
-     cordova-cli
-     cordova-android
-     cordova-ios
-     cordova-js
-   * If you don't want to use post-review tool, then on the web site:
-     * Click "New Review Request"
-     * On your workstation do a git diff and pipe the output to a file. In the new request,
select the git repo, the base dir where you did the diff, upload the diff file you created,
and click Create.
-     * On the next web page, fill in the description text, the name of the Branch (i.e.,
"master"), the Bug (i.e., "CB-4960"), the Description and the Testing Done fields. If you
want a review from a specific person, enter their userid / name in the People field. If you
want input from the whole community, enter "cordova" in the Groups field. Click the Submit
-   * After you have received sufficient feedback, click the button to mark your review as
Discarded or Submitted [to a stream].
-   * If you don't want to use ReviewBoard you can use Github Pull Request

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message