umberto-sonnino
commented
2 years ago lgtm. would be cool to put setup instructions for this into readme (im guessing there's a clang format thing in vscode too? going to look...)
Yeah, this actually ties nicely with VSCode so whenever you autoformat, it uses the settings in that .clang-format file I added.
I just ran it on our C++ folders like this:
cpp % find ./src -type f -name '*pp' | xargs clang-format -i
cpp % find ./include -type f -name '*.h' | xargs clang-format -i
cpp % find ./include -type f -name '*.hpp' | xargs clang-format -i But we shouldn't need to run it manually every time.
We could set a pre-commit hook that does it for us, though.
Just realized we didn't have the proper formatter set up in this repo, I figured it was worth a quick fix