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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
