<div dir="ltr"><div><div><div><div><div>Simon, Manuel,<br><br>any feedback about this? I see a few possible levels<br><br></div>1 Everything, bugfixes included<br></div>2 Every feature patch<br></div>3 Every patch to the new html/javascript code<br>
</div>4 Nothing, leave it to the contributor willingness<br><br></div>I'm opposed to 4 :) I tend to think we should do 2, because a lot of new code is landing and the more code without tests we need to maintain the worst the quality situation will get. I guess 3 would also be a possibility if we want to try it out and increase gradually.<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 13 May 2013 00:28, Daniel Narvaez <span dir="ltr"><<a href="mailto:dwnarvaez@gmail.com" target="_blank">dwnarvaez@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello,<div><br></div><div>I'd like to propose to make it a requirement, enforced by code reviews, to provide good test coverage when submitting new code. It will raise the bar for contributions but it's essential if we want to improve quality (and I think we have to). I can add a paragraph about it to sugar-docs, if we have consensus.</div>

<div><br></div><div>A few details<span></span>:</div><div><br></div><div>* What to do with patches which have been already submitted? I think it really depends on the patch, so I'd leave it to the reviewer discretion.</div>

<div>* Should this apply to bug fixes? I tend to think it should, we are not in a particularly active bug fixing period now, so it's a good time to start with those too.</div><div>* Cannot apply to javascript code yet, because the infra is not in place. Though writing the infra is on the short time priorities, so this should change soon.</div>

<div>* Cannot apply to activities because we are missing infra bits. It would not be too hard to add them, but I think we should focus on html activities now.</div><span class="HOEnZb"><font color="#888888"><br><br>-- <br>
Daniel Narvaez<br><br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>Daniel Narvaez<br>
</div>