Closed Yurlungur closed 2 years ago
There's currently a github actions outage, so the CI isn't triggering. Hopefully that will recover soon.
@brryan I think I've addressed all comments. I now:
format.sh
script so that it warns if clang-format is the wrong version. You can also point it to a specific binary with a command line variable. It also now errors out if there's no clang-format available.test_utils.hpp
header.
I think it's about time we formatted the repository, as it's getting pretty messy. This PR does the following:
bash/scripts/format.sh
, which queries git for every cpp and hpp file in git history, excluding submodules, and calls clang-format on all of them. Note a file must be committed to git for this script to format it.What do you all think? Are you happy with adding the workflow? Are you happy with the massive format commit? Does the script make sense to you? If you're happy, I think this should be a quick merge.