Add a clang-tidy GH action - #1661
Conversation
1233d3c to
f6a20c9
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
|
Please some description about what is going on here |
|
It's a mere test, I'm trying to add a clang-tidy GH action. I'm affraid it's going to take me some trials. Please unsubscribe from this draft. I'll set it as PR when ready. |
jmillan
changed the title
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
19 similar comments
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
To avoid clang-tidy complain about some header not being found which is required by sanitizer. See more here ZedThree/clang-tidy-review#157 (comment)
b58af74 to
785d083
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
2 participants
A github action that runs on every PR and executes clang-tidy on the changed files.
It also adds suggestions to the offending code which can be directly committed, although such suggestions are not formatted following .clang-format, hence
make formatmay need to be run locally and push the changes.