Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consider the use of clang-tidy as a linter #45

Open
willdurand opened this issue Feb 24, 2020 · 0 comments
Open

Consider the use of clang-tidy as a linter #45

willdurand opened this issue Feb 24, 2020 · 0 comments
Labels
code quality priority: p4 Issues at this priority level are "nice to have".

Comments

@willdurand
Copy link
Owner

I gave clang-tidy a try locally and that is interesting. It caught a few minor mistakes and gives good warnings. I see it as a linter/static analysis tool to prevent some mistakes.

We'll adopt clang-format to enforce stylistic conventions. clang-tidy is not about style but errors that can be caught with static analysis so having both would make sense.

That's not high priority, though.

@willdurand willdurand changed the title Consider using clang-tidy Consider the user of clang-tidy as a linter Feb 24, 2020
@willdurand willdurand changed the title Consider the user of clang-tidy as a linter Consider the use of clang-tidy as a linter May 10, 2020
@willdurand willdurand added the priority: p4 Issues at this priority level are "nice to have". label Dec 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code quality priority: p4 Issues at this priority level are "nice to have".
Projects
None yet
Development

No branches or pull requests

1 participant