This is Guideline reflect our current policy for core/protocols/devel libraries and unit test code. Make sure you check thing below before merging code to master.
Source code
NOTE: It is now required that all changes be submitted as pull requests, and approved by a second community member. All of the recommendations below still apply.
Do not commit code containing direct output to std::cout/cerr streams in to core/protocols/devel libraries or unit test. Use core::util::Tracer instead.
Use tabs rather than spaces for indentation and remove trailing whitespace. Also make sure you follow the other Coding Conventions such as removing "using" statements from header files.
Building
Test your build before committing it. (Replace '-j8' with number appropriate for your system, i.e. number of processors).
Make sure that everything is compiling and your modification did not produce any compilation warnings.
Unit test
Run unit test before committing any code, and make sure that your modifications did not break any tests.
If your changes break tests, fix them after insuring that the results are correct. If you can't fix some of the test or aren't sure if results correct, contact the developer responsible for this test and ask for help.
Integration Test
After commit
Additional information about our current coding guideline can be found here
Information on Commenting Guidelines (TODO: How do you even get access to this page?): https://www.rosettacommons.org/internal/doc/Rosetta++CommentingGuidelines.html