[llvm-dev] merge_guards_bot reports clang-tidy/clang-format findings unrelated to modified code

Fangrui Song via llvm-dev llvm-dev at lists.llvm.org
Thu Jan 2 14:19:16 PST 2020


First, I love this bot:) It makes LLVM's "pushing to master" practise less awful:)
Now the main topic...

> clang-tidy: fail. Please fix clang-tidy findings.

For example, on https://reviews.llvm.org/D72103#1801916 ,
merge_guards_bot reports clang-tidy findings of existing code, not just
the modified code.

I think https://github.com/google/llvm-premerge-checks should use
--line-filter= as clang-tidy/tool/clang-tidy-diff.py does.

> clang-format: fail. Please format your changes with clang-format by running git-clang-format HEAD^ or applying this patch.

Similarly, clang-format should use --lines to filter out untouched code.


More information about the llvm-dev mailing list