Great, Jason and I were just talking about this today.<div><br></div><div><proposed></div><div>One possible process flow we were discussing is that module leads still do their work from a forked repository like any other contributor. When changes are ready, the module lead issues a pull request, giving the integration managers (namely Pete) the notification/heads-up to review the code. The module lead then completes their own pull request, only deferring if changes are requested by the reviewer.</div>
<div><br></div><div>The reason for this process is so that all changes can be tracked back to pull requests, and those pull requests can be linked to the related JIRA.</div><div><br></div><div>Of course, there is always the exception that "hot changes" are made to the upstream master for non-normative changes (formatting, layout, pom parent versions, etc). Use at your discretion would be the general policy there.</div>
<div></proposed></div><div><div><br></div><div>Reasonable? Not reasonable?</div><div><br></div><div>-Dan<br><br><div class="gmail_quote">On Fri, Oct 22, 2010 at 6:19 AM, Pete Muir <span dir="ltr"><<a href="mailto:pmuir@bleepbleep.org.uk">pmuir@bleepbleep.org.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">I have set up teams for each module, giving the lead permission to push to their repo (except in a couple of cases where I couldn't find the leads username, so if you can't push, then please tell me your username). You should now be able to handle pull request directly.<br>
<br>
Let's see how this system works :-)<br>
_______________________________________________<br>
seam-dev mailing list<br>
<a href="mailto:seam-dev@lists.jboss.org">seam-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/seam-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/seam-dev</a><br>
</blockquote></div><br><br clear="all"><br>-- <br>Dan Allen<br>Principal Software Engineer, Red Hat | Author of Seam in Action<br>Registered Linux User #231597<br><br><a href="http://mojavelinux.com" target="_blank">http://mojavelinux.com</a><br>
<a href="http://mojavelinux.com/seaminaction" target="_blank">http://mojavelinux.com/seaminaction</a><br><a href="http://www.google.com/profiles/dan.j.allen" target="_blank">http://www.google.com/profiles/dan.j.allen</a><br>
</div></div>