<div dir="ltr">So am investigat[0]:<div>1. Cu submodules nu merge fiind ca sunt gist-uri si github nu le link-uie frumos</div><div>2. Nu merge sa lasi comment-uri decat pe commit-uri, nu fix pe fisiere. So ar insemna sa avem gen 1 commit/candidat.</div><div>Asa ca eu votez pentru a lasa sistem-ul asa cum l-a gandit Dan Serban. Pare util sa poti sa lasi comment-uri pentru alti revieweri, dar cred ca ne trebuie un sistem de code review dedicat, de investigat pentru la anu.</div><div><br></div><div>[0] - <a href="https://github.com/tudalex/webdev-2014-code-review/commit/086cc465fa262a6ac3324e0dba97a38d8668a54a">https://github.com/tudalex/webdev-2014-code-review/commit/086cc465fa262a6ac3324e0dba97a38d8668a54a</a></div></div><div class="gmail_extra"><br><div class="gmail_quote">2014-09-26 21:12 GMT+03:00 Dan Serban <span dir="ltr"><<a href="mailto:dserban01@gmail.com" target="_blank">dserban01@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">>De acord, dar tocmai tu ai spus problema, faptul ca trebuie sa faci<br>
>update. Oricum mi se pare stupid sa tot tweak-uie solutia dupa ce ne-o<br>
>trimit, daca ii facem code review imediat ce o trimit? Nu cred ca e viabil<br>
>sa asteptam dupa ei sa isi tot modifice solutia. Just my 3.14159.. cents<br>
><br>
<br>
</span>Pune tu la punct un proof of concept in varianta gandita de tine si<br>
hai sa vedem cum merge.<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
webdev mailing list<br>
<a href="mailto:webdev@lists.rosedu.org">webdev@lists.rosedu.org</a><br>
<a href="http://lists.rosedu.org/listinfo/webdev" target="_blank">http://lists.rosedu.org/listinfo/webdev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Tudorică Constantin Alexandru<br><a href="http://tudalex.com" target="_blank">http://tudalex.com</a><br>
</div>