<div dir="ltr">Thanks Eric! I've used Reviewboard at a previous job and I'm fairly sure that it aligns better with the way the Juju Core team likes to work than Github's review features.<div><br></div><div>Two questions:</div><div><br></div><div>1. Is this what we're supposed to be doing from right now?</div><div><br></div><div>2. I'm pretty sure some configuration of the rbt tool is required so that it knows how to talk to the Reviewboard server. Is there a config file available? <br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 11 September 2014 03:58, Eric Snow <span dir="ltr"><<a href="mailto:eric.snow@canonical.com" target="_blank">eric.snow@canonical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Steps for a review of a PR:<br>
<br>
1. create pull request in github<br>
2. run "rbt post" while at your branch to create a review request [1][2]<br>
3. open the review request in your browser and "publish" it [3]<br>
4. add a comment to the PR with a link to the review request<br>
5. address reviews until you get a "Ship It!"<br>
6. add a $$merge$$ comment to the PR<br>
<br>
Both github and ReviewBoard support various triggers/hooks and both<br>
have robust HTTP APIs.  So we should be able to automate those steps<br>
(e.g. PR -> review request, "ship it" -> $$merge$$).  However, I don't<br>
see that automation as a prerequisite for switching over to<br>
ReviewBoard.<br>
<br>
Updating an existing review request:<br>
1. run "rbt post -u" (or the explicit "rbt post -r #")<br>
2. open the review request in your browser and "publish" it [3]<br>
<br>
FYI, Reviewboard supports chaining review requests.  Run rbt post<br>
--parent <parent branch>.<br>
<br>
I'll be updating the contributing doc relative to ReviewBoard (i.e.<br>
with the above info) once we settle in with the new tool.<br>
<br>
-eric<br>
<br>
[1] Make sure your branch is based on upstream master.  Otherwise this<br>
will not work right.<br>
[2] Reviewboard links revision IDs to review requests.  So if you<br>
already have a review request for a particular revision (e.g. your<br>
branch), then "rbt post" will fail.  Use "rbt post -u" or "rbt post -r<br>
#" instead.<br>
[3] rbt post has some options you should consider using:<br>
  - automatically publish the review request: rbt post --publish (or -p)<br>
  - open a browser window with the new review request: rbt post --open (or -o)<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Juju-dev mailing list<br>
<a href="mailto:Juju-dev@lists.ubuntu.com">Juju-dev@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/juju-dev" target="_blank">https://lists.ubuntu.com/mailman/listinfo/juju-dev</a><br>
</font></span></blockquote></div><br></div>