blazingmq icon indicating copy to clipboard operation
blazingmq copied to clipboard

CI: Add CI checks with clang-tidy

Open hallfox opened this issue 1 year ago • 1 comments

This adds a new check to our workflows using clang-tidy. Right now we only enable analyzer checks, though we could look at expanding to more in the future. We are primarily interested in getting another pass over our code checking for C++03 syntax compatibility, since we've had a few instances of modern syntax constructs getting into the code base. clang-tidy will do a bit more than syntax checks, but I believe having will be a boon overall.

hallfox avatar Oct 09 '24 20:10 hallfox

@hallfox

Work I am doing in the same vein as https://github.com/bloomberg/blazingmq/pull/706 is to make the clang-format, black, etc checks we do in CI first-class CMake targets. It may make sense to add a clang-tidy CMake target first, fix the issues it finds, then update this PR to use that CMake target.

Thoughts?

pniedzielski avatar Apr 23 '25 00:04 pniedzielski