[PATCH] D78985: [clang-tidy] NFC: Cleanup Python scripts

Tue Apr 28 07:29:07 PDT 2020

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