Some tweaks of the formatting machinery

lanl / phoebus

Phifty One Ergs Blows Up A Star

BSD 3-Clause "New" or "Revised" License

46 stars 2 forks source link

Some tweaks of the formatting machinery #90

Closed Yurlungur closed 2 years ago

Yurlungur commented 2 years ago

The CI format was passing no matter what. I removed the line making this happen. I also tweak the format.sh script for Ben's use-case. In particular:

Script always outputs the clang-format binary it found
Script always outputs the clang-format version. It's up to the user to discern whether or not the version is correct. I didn't want to have to parse every possible format string.

Yurlungur commented 2 years ago

@brryan are you happy for me to merge this? or is something still wrong?