Added pre-commit Action and pre-commit formatting fixes#40
Added pre-commit Action and pre-commit formatting fixes#40JeffGarland merged 4 commits intobemanproject:mainfrom
pre-commit Action and pre-commit formatting fixes#40Conversation
Yes, this branch branches directly off of main. You can verify by checking the parent hash (
I'm sorry, I didn't quite get what you mean. Could you clarify that a bit? |
|
Let's try to unblock this PR
I think what @neatudarius is saying is that because this PR is about pre-commit actions he didn't expect to see .cpp files modified. I believe the clarification here is that clang-format has been run on those files. I checked them over and it all appears to be whitespace changes consistent with what clang-format does (telling is the addition of the end namespace comments). I'm going to go ahead and merge this. |
As requested in #39, this adds a
pre-commitGitHub Action for commits tomainand PRs. The.pre-commit-config.yamland the GitHub Action script are taken verbatim frombemanproject/inplace-vector. (Here and here respectively, to be precise.)A large portion of the diff for this PR is simply the changes I had to make to appease the
pre-commitCI. I tried to be as minimally invasive to the README as possible.This PR, along with #39, addresses issue #3 entirely.