<div dir="ltr">Really, rbt pull -p is the only new step.  All the rest of that is stuff you should already be doing as a normal part of writing code and making pull requests.  I guess adding the link on the PR to the review is also a new step.  If you really want to count that as a step.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 15, 2014 at 10:50 AM, 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"><span class="">On Mon, Sep 15, 2014 at 8:09 AM, Eric Snow <<a href="mailto:eric.snow@canonical.com">eric.snow@canonical.com</a>> wrote:<br>
> Yeah, those steps are a lot, though keep in mind that effectively it's<br>
> only 2 steps more than before if you use the -p flag to rbt post and<br>
> were already keeping your local master up to date.<br>
<br>
</span>Just to be clear, here are the steps again, slightly reformatted:<br>
<br>
(0). Rebase relative to upstream master.<br>
  - if origin is different than upstream, sync and push it<br>
1. Create a pull request via github.<br>
2. Run "rbt pull -p" while at your branch to create a review request.<br>
<span class="">3. add a comment to the PR with a link to the review request.<br>
4. address reviews until you get a "Ship It!" (like normal, with LGTM).<br>
</span><span class="">5. add a $$merge$$ comment to the PR (like normal).<br>
</span>6. mark the review request as submitted.<br>
<br>
So, steps 2, 3, and 6 are completely new.  They don't add a lot of<br>
work and I plan on automating all 3 of those new steps.<br>
<br>
Step (0) is also pretty easy and I'll argue that people should be<br>
doing it anyway.<br>
<div class="HOEnZb"><div class="h5"><br>
-eric<br>
<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>
</div></div></blockquote></div><br></div>