kbobyrev added a comment. 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. ================ Comment at: clang-tools-extra/clang-tidy/add_new_check.py:1 -#!/usr/bin/env python +#!/usr/bin/env python3 # ---------------- hokein wrote: > we should keep using `python` I think. Oops, a leftover from testing, thanks for noticing! 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
