Skip to content

test: do not lint macros files (again)

Rodrigo Muino Tomonari requested to merge github/fork/cclauss/Python-macros-again into master

This PR is #24885 but without the modifications to deps/v8/src/js/macros.py

These three .py files do not contain any Python code. Instead they contain "macros" which are processed by js2c.py. Our Python linter (make lint-py) will properly flag these files as containing Python syntax errors. One solution would be to change the filename suffixes (.py -->.js2c) or we could add a linter directive (# noqa) to the comments of each file to tell our linter to ignore those files. This PR implements the latter.

@targos Your review please.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

Merge request reports