Re: github code reviews

2014-03-10 Thread Richard Harding
On Mon, 10 Mar 2014, Jonathan Aquilina wrote:

> I have two suggestions that might help, but first let me tell you a bit
> about myself.
> 
> I am project manager for LMMS (Linux Multimedia Studio) and we recently
> started using github for our version control hosting.
> 
> With this we started using continuous integration provided free of charge
> by Travis CI. It is very easy to integrate into a github repo by just
> adding a YAML file if I am not mistaken.  Once that is in place every
> commit triggers a build on the Travis CI build machines. It as well on the
> pull requests tells you if the build passes. If it does not it also alerts
> the committer that the build has failed so that it can quickly be fixed.

We originally looked at this for the Juju GUI but we could not use it.
Travis does not support the idea of a 'landing test run'. If your code
review takes a few days, the trunk that you ran in Travis could be changed
enough that your code no longer passes CI. This is why we went with Jenkins
and wrote the lander [2] tool. It forces a final CI pass and automates the
landing to the trunk branch.

I did reach out to the Travis folks and their reply was that they did not
think this idea of the post-review test/land work was part of their mission
and were not interested in adding support for it. I'm not sure how they
have that logic myself though.


[2] https://github.com/juju/jenkins-github-lander

Rick



-- 
Juju-dev mailing list
Juju-dev@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju-dev


Re: github code reviews

2014-03-10 Thread Jonathan Aquilina
I have two suggestions that might help, but first let me tell you a bit
about myself.

I am project manager for LMMS (Linux Multimedia Studio) and we recently
started using github for our version control hosting.

With this we started using continuous integration provided free of charge
by Travis CI. It is very easy to integrate into a github repo by just
adding a YAML file if I am not mistaken.  Once that is in place every
commit triggers a build on the Travis CI build machines. It as well on the
pull requests tells you if the build passes. If it does not it also alerts
the committer that the build has failed so that it can quickly be fixed.

If you want to take it down another router in addition to this.
Libreoffice uses gerrit which is a code review system, which has a web
based interface to manage the patches for review.

Regards
Jonathan

> On Fri, Mar 7, 2014 at 4:07 PM, Nate Finch 
> wrote:
>
>> I like github.  I don't like in-line diffs and one email per code review
>> comment, which is all github provides.  So I did a quick google, and
>> this
>> came up:
>>
>> https://review.gerrithub.io/#/c/2160/2/roles/packstack/rdo/tasks/main.yml
>>
>>
>> it looks pretty good.   http://gerrithub.io/
>>
>> --
>> Juju-dev mailing list
>> Juju-dev@lists.ubuntu.com
>> Modify settings or unsubscribe at:
>> https://lists.ubuntu.com/mailman/listinfo/juju-dev
>>
>>
> There is also https://github.com/jarodwen/git-cl which many are claiming
> to
> have good luck with. It would mimic or existing work flow and continue to
> use Rietveld.
> --
> Juju-dev mailing list
> Juju-dev@lists.ubuntu.com
> Modify settings or unsubscribe at:
> https://lists.ubuntu.com/mailman/listinfo/juju-dev
>



-- 
Juju-dev mailing list
Juju-dev@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju-dev


Re: github code reviews

2014-03-09 Thread Wayne Witzel III
On Fri, Mar 7, 2014 at 4:07 PM, Nate Finch  wrote:

> I like github.  I don't like in-line diffs and one email per code review
> comment, which is all github provides.  So I did a quick google, and this
> came up:
>
> https://review.gerrithub.io/#/c/2160/2/roles/packstack/rdo/tasks/main.yml
>
>
> it looks pretty good.   http://gerrithub.io/
>
> --
> Juju-dev mailing list
> Juju-dev@lists.ubuntu.com
> Modify settings or unsubscribe at:
> https://lists.ubuntu.com/mailman/listinfo/juju-dev
>
>
There is also https://github.com/jarodwen/git-cl which many are claiming to
have good luck with. It would mimic or existing work flow and continue to
use Rietveld.
-- 
Juju-dev mailing list
Juju-dev@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju-dev


Re: github code reviews

2014-03-07 Thread Richard Harding
On Fri, 07 Mar 2014, Nate Finch wrote:

> I like github.  I don't like in-line diffs and one email per code review
> comment, which is all github provides.  So I did a quick google, and this
> came up:
>
> https://review.gerrithub.io/#/c/2160/2/roles/packstack/rdo/tasks/main.yml
>
>
> it looks pretty good.   http://gerrithub.io/

There's a chrome extension for enabling side by side diffs I've been using
as well.

https://chrome.google.com/webstore/detail/octosplit/mnkacicafjlllhcedhhphhpapmdgjfbb


--

Rick Harding

Juju UI Engineering
https://launchpad.net/~rharding
@mitechie

-- 
Juju-dev mailing list
Juju-dev@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju-dev


Re: github code reviews

2014-03-07 Thread Matt Bruzek
Hi Nate,

We used ReviewBoard at my last area and all the developers really liked the
tool.  It was very easy to review, and comment on code.

ReviewBoard ( www.reviewboard.org ) allows in-line comments and according
to the web page works with Bazaar and Git.

I found the interface very clean and easy to understand.

   - Matt


On Fri, Mar 7, 2014 at 3:07 PM, Nate Finch  wrote:

> I like github.  I don't like in-line diffs and one email per code review
> comment, which is all github provides.  So I did a quick google, and this
> came up:
>
> https://review.gerrithub.io/#/c/2160/2/roles/packstack/rdo/tasks/main.yml
>
>
> it looks pretty good.   http://gerrithub.io/
>
> --
> Juju-dev mailing list
> Juju-dev@lists.ubuntu.com
> Modify settings or unsubscribe at:
> https://lists.ubuntu.com/mailman/listinfo/juju-dev
>
>
-- 
Juju-dev mailing list
Juju-dev@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju-dev