llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-tidy Author: Baranov Victor (vbvictor) <details> <summary>Changes</summary> This checks seem to give too many FP, disabling it for now. E.g. it wants to always remove `SmallString.h` even if the header is actually needed. --- Full diff: https://github.com/llvm/llvm-project/pull/221417.diff 1 Files Affected: - (modified) clang-tools-extra/clang-tidy/.clang-tidy (+1-1) ``````````diff diff --git a/clang-tools-extra/clang-tidy/.clang-tidy b/clang-tools-extra/clang-tidy/.clang-tidy index 38caf5ccb555c..8ba9f07074354 100644 --- a/clang-tools-extra/clang-tidy/.clang-tidy +++ b/clang-tools-extra/clang-tidy/.clang-tidy @@ -19,7 +19,7 @@ Checks: > google-readability-casting, misc-const-correctness, -misc-explicit-constructor, - misc-include-cleaner, + -misc-include-cleaner, modernize-*, -modernize-avoid-c-arrays, -modernize-pass-by-value, `````````` </details> https://github.com/llvm/llvm-project/pull/221417 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
