Here's a list of things we might do to reduce PR bickering and improve software correctness. I think they're not worth it now (except black and maybe basic CI) but here's a list anyways:
[ ] Automatic formatting
[x] Python (black)
[ ] C++ (clang-format)
[ ] Coverage reports
Especially: compare coverage reports in tests vs. experiments. If significant parts of the experiment coverage aren't covered by tests, we're in trouble
[x] Continuous integration: Automatically run tests before merging every PR
[x] Build with clang (faster, more warnings, as a downside IDK if valgrind works)
Here's a list of things we might do to reduce PR bickering and improve software correctness. I think they're not worth it now (except
black
and maybe basic CI) but here's a list anyways:[ ] Automatic formatting
black
)clang-format
)[ ] Coverage reports
[x] Continuous integration: Automatically run tests before merging every PR
[x] Build with clang (faster, more warnings, as a downside IDK if valgrind works)