-
Notifications
You must be signed in to change notification settings - Fork 15.4k
[clang-tidy][NFC] Add empty '.clang-tidy' file in tests dir to silent warnings in IDE #171029
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
Conversation
|
@llvm/pr-subscribers-clang-tools-extra Author: Baranov Victor (vbvictor) ChangesWhen working on tests, Full diff: https://github.com/llvm/llvm-project/pull/171029.diff 1 Files Affected:
diff --git a/clang-tools-extra/test/clang-tidy/.clang-tidy b/clang-tools-extra/test/clang-tidy/.clang-tidy
new file mode 100644
index 0000000000000..612bd0ee8de8a
--- /dev/null
+++ b/clang-tools-extra/test/clang-tidy/.clang-tidy
@@ -0,0 +1 @@
+Checks: '-*'
|
|
@llvm/pr-subscribers-clang-tidy Author: Baranov Victor (vbvictor) ChangesWhen working on tests, Full diff: https://github.com/llvm/llvm-project/pull/171029.diff 1 Files Affected:
diff --git a/clang-tools-extra/test/clang-tidy/.clang-tidy b/clang-tools-extra/test/clang-tidy/.clang-tidy
new file mode 100644
index 0000000000000..612bd0ee8de8a
--- /dev/null
+++ b/clang-tools-extra/test/clang-tidy/.clang-tidy
@@ -0,0 +1 @@
+Checks: '-*'
|
|
This is mostly just a hypothesis for now. I think the Windows build fails because On Windows, the argument is parsed as a check named Previously, the test passed because it fell back to the root I opened #171033 to fix this. @vbvictor can you try manually applying the fix to |
|
Will be good idea to change commit summary to end with |
Yes, also missed that my title was bad initially. |
When working on tests,
clangdwith option--clang-tidywill report warnings from root clang-tidy config.I believe these warnings serve no purpose in tests, so we better disable them to silent warnings in IDE.