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

[clang-tidy] Allow to override checker list #3203

Merged

Conversation

csordasmarton
Copy link
Contributor

@csordasmarton csordasmarton commented Feb 22, 2021

Closes #1798

If the file specified by the --tidy-config option contains a Checks
key or the Checks option is specified through the --analyzer-config
option the -checks option will be skipped from the actual clang-tidy
analyzer command, so only the checkers what specified in the Checks
key will be used on analysis.

If the clang-tidy:take-config-from-directory=true option is set, use
the .clang-tidy configuration files and skip setting the -checks
parameter of the clang-tidy binary.

@csordasmarton csordasmarton added WIP 💣 Work In Progress analyzer 📈 Related to the analyze commands (analysis driver) usability 👍 Usability-related features labels Feb 22, 2021
@csordasmarton csordasmarton force-pushed the overridable_clang_tidy_checks branch from b5bdf37 to 738b9f1 Compare April 7, 2021 11:41
@csordasmarton csordasmarton removed the WIP 💣 Work In Progress label Apr 7, 2021
@csordasmarton csordasmarton force-pushed the overridable_clang_tidy_checks branch from 738b9f1 to 9a3c6a9 Compare April 7, 2021 12:14
@csordasmarton csordasmarton added this to the release 6.17.0 milestone Jul 13, 2021
If the file specified by the `--tidy-config` option contains a `Checks`
key or the `Checks` option is specified through the `--analyzer-config`
option the `-checks` option will be skipped from the actual clang-tidy
analyzer command, so only the checkers what specified in the `Checks`
key will be used on analysis.

If the `clang-tidy:take-config-from-directory=true` option is set, use
the `.clang-tidy` configuration files and skip setting the `-checks`
parameter of the clang-tidy binary.
@csordasmarton csordasmarton force-pushed the overridable_clang_tidy_checks branch from 9a3c6a9 to a083726 Compare August 17, 2021 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analyzer 📈 Related to the analyze commands (analysis driver) usability 👍 Usability-related features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

--tidy-config checks are ignored by clang-tidy
2 participants