----- Original Message -----
> Finally, we have a working, if incomplete, set of unit tests in our
> project. Thanks, Saggi (and others!).
> 
> Every module should have comprehensive unit tests. Writing good unit
> tests can be hard, and writing testable code is even harder. However,
> I
> would like to request that every new module, and every new
> functionality
> have both: testable code and a unit test that proves it.
> 
> In my opinion this should be non-optional. I hope anybody nacks a
> patch
> with no accompanying unit test, before I'm tempted to accept it.
> 
> In a less dramatic tone, I wish new files are added to pep8's
> whitelist, so that we advance towards clearer code in Vdsm.
> 
> What do you say? Can this be a "formal" policy?

+2 :)

> 
> Regards,
> Dan.
> _______________________________________________
> vdsm-devel mailing list
> vdsm-devel@lists.fedorahosted.org
> https://fedorahosted.org/mailman/listinfo/vdsm-devel
> 
_______________________________________________
vdsm-devel mailing list
vdsm-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/vdsm-devel

Reply via email to