[PATCH] D78985: [clang-tidy] NFC: Cleanup Python scripts
Eugene Zelenko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 28 08:02:01 PDT 2020
Eugene.Zelenko added a comment.
In D78985#2007715 <https://reviews.llvm.org/D78985#2007715>, @kbobyrev wrote:
> In D78985#2007669 <https://reviews.llvm.org/D78985#2007669>, @Eugene.Zelenko wrote:
>
> > It would be great to also run Flake9 and PyLint.
>
>
> Those were actually from pylint. That's a good idea, but I didn't fix all the warnings there and while I might get back to this in the future I don't think it's in the scope of this patch as this is a pretty trivial NFC cleanup.
If number of PyLint warnings (excluding identifiers naming) is not too big, it may make sense to clean-up in one commit.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78985/new/
https://reviews.llvm.org/D78985
More information about the cfe-commits
mailing list