search

lpenz / omnilint

Statically analyse any file with the appropriate tools
MIT License
3 stars 0 forks source link

Add clang-format/clang-tidy #5

Open jayvdb opened 3 years ago

jayvdb commented 3 years ago

This is just a suggestion.

We are currently using https://github.com/DoozyX/clang-format-lint-action because it allows specifying the clang version, and we're a bit stuck on clang-6 as it was the most easily installable at the time, and was still available up to ubuntu focal https://packages.ubuntu.com/focal/clang-format-6.0

https://github.com/swift-nav/libsettings/blob/master/.github/workflows/lint.yml

lpenz commented 3 years ago

clang-format can be a nice addition. I do use DoozyX/clang-format-lint-action nowadays too.

clang-tidy is actually supported as a preset by lpenz/ghaction-cmake. It fits better there because it's supposed to be invoked via the build system, as a compiler.

jayvdb commented 2 years ago

I notice that clang-format is installed in https://github.com/lpenz/ghaction-cmake , but there is no preset for clang-format there. It seems appropriate to add a thin wrapper for clang-format there which just checks formatting of all relevant files.