Return-Path: X-Original-To: apmail-mesos-commits-archive@www.apache.org Delivered-To: apmail-mesos-commits-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id E1B7017596 for ; Wed, 11 Mar 2015 01:39:59 +0000 (UTC) Received: (qmail 44144 invoked by uid 500); 11 Mar 2015 01:39:54 -0000 Delivered-To: apmail-mesos-commits-archive@mesos.apache.org Received: (qmail 44057 invoked by uid 500); 11 Mar 2015 01:39:54 -0000 Mailing-List: contact commits-help@mesos.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@mesos.apache.org Delivered-To: mailing list commits@mesos.apache.org Received: (qmail 43981 invoked by uid 99); 11 Mar 2015 01:39:54 -0000 Received: from eris.apache.org (HELO hades.apache.org) (140.211.11.105) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 11 Mar 2015 01:39:54 +0000 Received: from hades.apache.org (localhost [127.0.0.1]) by hades.apache.org (ASF Mail Server at hades.apache.org) with ESMTP id 8FA94AC0703 for ; Wed, 11 Mar 2015 01:39:54 +0000 (UTC) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r1665754 [3/3] - in /mesos/site: publish/documentation/latest/mesos-developers-guide/index.html publish/documentation/mesos-developers-guide/index.html publish/sitemap.xml source/documentation/latest/mesos-developers-guide.md Date: Wed, 11 Mar 2015 01:39:53 -0000 To: commits@mesos.apache.org From: dlester@apache.org X-Mailer: svnmailer-1.0.9 Message-Id: <20150311013954.8FA94AC0703@hades.apache.org> Modified: mesos/site/source/documentation/latest/mesos-developers-guide.md URL: http://svn.apache.org/viewvc/mesos/site/source/documentation/latest/mesos-developers-guide.md?rev=1665754&r1=1665753&r2=1665754&view=diff ============================================================================== --- mesos/site/source/documentation/latest/mesos-developers-guide.md (original) +++ mesos/site/source/documentation/latest/mesos-developers-guide.md Wed Mar 11 01:39:53 2015 @@ -46,8 +46,8 @@ layout: documentation 9. Make sure to pull in any changes that have been committed to master branch. Using Git, do this via something like: 1. `git checkout master` 2. `git pull` - 3. `git co my_branch` - 4. Check the output of `git diff master` and make sure it lists only your changes. If other changes you did not make are listed, try a rebase to bring your branch up to date with master. + 3. `git checkout my_branch` + 4. Check the output of `git diff master` and make sure it lists only your changes. If other changes you did not make are listed, try `git rebase master` to bring your branch up to date with master. ### Submit your patch 1. You're ready to submit your patch for review! @@ -55,10 +55,11 @@ layout: documentation 2. The easiest (and recommended) way to submit reviews is through `post-reviews.py` a wrapper around post-review. 3. First, install post-review. [See Instructions](http://www.reviewboard.org/docs/manual/dev/users/tools/post-review/) 4. Configure post-review. The easiest method is to symlink to the sample config: ln -s support/reviewboardrc .reviewboardrc - 5. From your local branch run `support/post-reviews.py`. - 6. Note that `post-reviews.py` creates a new review for every commit on your branch that is different from the `master`. - 7. Be sure to add your JIRA issue id (e.g. MESOS-01) to the field labeled "Bugs" (this will automatically link) - 8. Under "Description" in addition to details about your changes, include a description of any wiki documentation pages need to be added, or are affected by your changes (e.g. did you change or add any configuration options/flags? Did you add a new binary?) + 5. Log into Review Board from the command line: run `rbt status` + 6. From your local branch run `support/post-reviews.py`. + 7. Note that `post-reviews.py` creates a new review for every commit on your branch that is different from the `master`. + 8. Be sure to add your JIRA issue id (e.g. MESOS-01) to the field labeled "Bugs" (this will automatically link) + 9. Under "Description" in addition to details about your changes, include a description of any documentation pages that need to be added, or are affected by your changes (e.g. did you change or add any configuration options/flags? Did you add a new binary?) 2. Wait for a code review from another Mesos developer via Review Board, address their feedback and upload updated patches until you receive a "Ship It" from a Mesos committer. 1. When addressing feedback, adjust your existing commit(s) instead of creating new commits, otherwise `post-reviews.py` will create a new review (`git rebase -i` is your friend).