[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