richlowe's been working on both, but neither are in use with OI.<br><br>My preference for all of this functionality would be to have the validation done by the pull request. Webrevs could be generated automatically, and the applicable nits would give feedback requesting either that they be resolved for a subsequent pull or by providing quick notes or tick boxes to explain why they aren't strictly necessary. You're provided an issue tracker ID for the submit request, and that tracks these kinds of things and is responsible for generating the webrev and sending out the review mail once issues immediately applicable to the submission are resolved. Workflow would also be responsible for checking CI build and test status as a prereq. An issue tracker could used to provide all the workflow state, but if that's too clunky (or the issue tracker too unreliable), it could be put into a database.<br>
<br>That's my cocktail napkin sketch of workflow (or at least the basics--there are other pieces I have in mind), which fuses some stuff I talked about at the illumos hackathon with the floor discussion on the contribution process we had at FOSDEM.<br>
<br><div class="gmail_quote">On Sun, Feb 12, 2012 at 6:27 PM, Milan Jurik <span dir="ltr"><<a href="mailto:milan.jurik@xylab.cz">milan.jurik@xylab.cz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Hi Alan,<br>
<br>
Alan Coopersmith píše v ne 12. 02. 2012 v 10:01 -0800:<br>
<div class="im">> On 02/12/12 09:29 AM, Milan Jurik wrote:<br>
> > Btw. how is status of cadmium features on git? Like recommit?<br>
><br>
> git rebase is far more powerful & useful than cadmium recommit,<br>
> especially once you learn git rebase --interactive.<br>
><br>
<br>
</div>I know rebase is very powerfull but as "recommit" is key part of the<br>
process, rebase power is also danger. recommit is simple enough to be<br>
nearly 100% safe to use by everybody.<br>
<div class="im"><br>
> The cadmium features that are missing from git are the ones more<br>
> specific to the opensolaris project, such as webrev integration<br>
> and checking project policy (nits, pbchk, etc.).<br>
><br>
<br>
</div>I think there is support for git in webrev in queue. pbchk support<br>
probably not yet.<br>
<br>
Best regards,<br>
<br>
Milan<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
_______________________________________________<br>
oi-dev mailing list<br>
<a href="mailto:oi-dev@openindiana.org">oi-dev@openindiana.org</a><br>
<a href="http://openindiana.org/mailman/listinfo/oi-dev" target="_blank">http://openindiana.org/mailman/listinfo/oi-dev</a><br>
</div></div></blockquote></div><br>