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

turn on flags #11

Open
sualehasif opened this issue Jan 5, 2022 · 3 comments
Open

turn on flags #11

sualehasif opened this issue Jan 5, 2022 · 3 comments

Comments

@sualehasif
Copy link

Hi, how can you turn on --fix with clang-tidy

@erenon
Copy link
Owner

erenon commented Jan 5, 2022

You can't directly, because of parallel execution (clang-tidy would corrupt the files). You have to merge the fix files first, and apply them in one go, as described in the clang-tidy docs. Look for clang-apply-replacements

@sualehasif
Copy link
Author

sualehasif commented Jan 5, 2022

Mmm, thanks for this. That works for me now. I still get a number of errors on *.h files and requires renaming them to *.hpp which is not in my style. I know you can use the flag -assume-filename=.hxx but not sure how to pass that to bazel-clang-tidy.

@sualehasif
Copy link
Author

Also, is it possibly to skip apply clang-tidy to some of the files in the build?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants