<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div>I'm sorry if I came across as though I was suggesting to entirely change digium's workflow earlier, that wasn't what I meant at all.</div>
<div><br></div><div>When I talked about using GitHub I didn't mean that you would have to use all of the features like wiki/issues etc, only that having the code in a public place (and easy to view and use) would only benefit the projects - <a href="http://git.asterisk.org">git.asterisk.org</a> is not a nice front end to git, it's the standard web GUI on top of git.</div>
<div><br></div><div>I also mentioned using bitbucket, is this something that digium have looked at? It's part of atlassians product range which is fully integrated into jira/confluence etc. </div><div><br></div><div>How-ever patches etc are done, fully depends on what's good for the project and if pull requests don't fit due to the license agreements then that's entirely up to the project, but the key thing here is engaging the community via one of these platforms; GitHub is the home of open source projects and will only invite people to fork and patch things, even if they then have to then submit those patches through jira.</div>
<div><br></div><div>If no-one has taken a look at bitbucket then it's something that digium should be looking at - its part of atlassians product line that is integrated so fully into digium's workflows.</div><div>
<br></div><div>The great thing about git is its distributed nature, take the bits that work best for the project,</div><div><br>Dan</div><div><br>On Dec 6, 2012, at 18:04, Russell Bryant <<a href="mailto:russell@russellbryant.net">russell@russellbryant.net</a>> wrote:<br>
<br></div><blockquote type="cite"><div>On Thu, Dec 6, 2012 at 12:44 PM, Shaun Ruffell <span dir="ltr"><<a href="mailto:sruffell@digium.com" target="_blank">sruffell@digium.com</a>></span> wrote:<br><div class="gmail_extra">
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Thu, Dec 06, 2012 at 12:02:13PM -0500, Russell Bryant wrote:<br>
> On Thu, Dec 6, 2012 at 11:37 AM, Shaun Ruffell <<a href="mailto:sruffell@digium.com">sruffell@digium.com</a>> wrote:<br>
><br>
</div><div class="im">> > - Gerrit is still in my queue to demo to see if it will solve the<br>
> > issue so I don't have a comment there.<br>
><br>
</div><div class="im">> I've been using gerrit a lot lately and I like it quite a bit. Some<br>
> interesting notes:<br>
><br>
> 1) It does support posting a patch series. The UI isn't perfect for it,<br>
> but it's there. I expect that to be improved in gerrit in the future given<br>
> the git-centric nature of gerrit and the importance of patch series for<br>
> projects using git.<br>
><br>
> Here is one example:<br>
> <a href="https://review.openstack.org/#/q/project:openstack/nova+branch:master+topic:bp/nova-compute-cells,n,z" target="_blank">https://review.openstack.org/#/q/project:openstack/nova+branch:master+topic:bp/nova-compute-cells,n,z</a><br>
><br>
> If you go into a specific patch, you will see a "Dependencies" section,<br>
> where you can see the patch(es) that come before and/or after the one<br>
> you're looking at.<br>
<br>
</div>Intersting. So you can list those dependencies for each patch but<br>
can reviewers accept / reject / comment on the entire series as a<br>
whole or does each individual patch need to be touched/approved?</blockquote><div><br></div><div>Each individual patch needs to be approved. You can't review/approve the series as a whole.</div><div><br></div><div>There's a lot of room for improvement in the support for a patch series, but at least it exists in some form. I haven't seen anything better yet.</div>
<div><br></div><div>(Sorry, email doesn't count. :-p )</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
> 2) The commit message is reviewed just like the source changes in a patch.<br>
> Again, take a look at any patch above as an example.<br>
><br>
> 3) Gerrit has some built-in CLA checking logic that can be hooked into.<br>
> <a href="http://review.openstack.org" target="_blank">review.openstack.org</a> uses it. You must have a CLA to be able to push<br>
> changes there, which is the only path for getting patches in.<br>
<br>
</div>I probably need to bump up installing / using Gerrit on my priority<br>
list. One thing that I still think I'll miss is the ability to<br>
review code offline but I have a feeling I'm the only one who really<br>
likes to do that...</blockquote><div><br></div><div>It's at least easy enough to read the code offline. Each review gives you the git commands to fetch the changes into your local repo. Making inline comments requires online access though AFAIK. <br>
</div><div><br></div><div>-- </div><div>Russell Bryant</div></div></div>
</div></blockquote><blockquote type="cite"><div><span>--</span><br><span>_____________________________________________________________________</span><br><span>-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com">http://www.api-digital.com</a> --</span><br>
<span></span><br><span>asterisk-dev mailing list</span><br><span>To UNSUBSCRIBE or update options visit:</span><br><span> <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev">http://lists.digium.com/mailman/listinfo/asterisk-dev</a></span></div>
</blockquote></body></html>