Den 2018-04-16 kl. 22:46, skrev Dominik 'Rathann' Mierzejewski:
On Monday, 16 April 2018 at 20:20, David van der Spoel wrote:
Den 2018-04-16 kl. 17:36, skrev David Koes:
I didn't chime in since I thought it was obviously a good idea.
However, I strongly agree that the process of creating a test case needs
to be as simple and documented as possible. I had a test case with my
last pull request, but it required a fair amount of poking around to
figure out how to best implement it (and this experience prompted the
GSoC project).
Also, test cases may not make sense for some pull requests (e.g.
documentation).
Agree tests are a must.
How about making warning-free code a must?
Warning-free under which compiler (and version)? GCC adds new warnings
in every release.
Under all compilers. Obviously we can only fix the warnings we are getting.
We have this policy in http://www.gromacs.org and it is enforced
automatically by the robot that verifies patches. No patches that
produce warnings on any platform (Linux, Mac, Windows) will be accepted.
Regards,
Dominik
--
David van der Spoel, Ph.D., Professor of Biology
Head of Department, Cell & Molecular Biology, Uppsala University.
Box 596, SE-75124 Uppsala, Sweden. Phone: +46184714205.
http://www.icm.uu.se
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
OpenBabel-Devel mailing list
OpenBabel-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbabel-devel
