[llvm-dev] clang-tidy pre-merge checks in Phabricator not working?

[Wang, Pengfei via llvm-dev]

Did you have clang-format in your path when you committed your patch to Phabricator? I've observed no Lint remarks in Phabricator reviews if I didn’t set the path.

Thanks
Phoebe (Pengfei)

From: llvm-dev <llvm-dev-bounces at lists.llvm.org> On Behalf Of James Henderson via llvm-dev
Sent: Monday, November 22, 2021 5:06 PM
To: llvm-dev <llvm-dev at lists.llvm.org>
Subject: [llvm-dev] clang-tidy pre-merge checks in Phabricator not working?

Hi,

No idea who to direct this towards, but we at one point had clang-tidy linter remarks in Phabricator reviews, just like we have clang-format notes. However, I've seen several cases in recent reviews where clang-tidy hasn't complained about violations (specifically to do with function and variable name casing). Has this been disabled deliberately or is it supposed to be working, but isn't for some reason?

James
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20211122/50db6a34/attachment.html>