A flake8 [1] linter was landed to central a couple weeks ago, and now
there is a corresponding taskcluster task that uses it. It only lints
directories that are explicitly whitelisted. To start, these are:

* python/mozlint
* tools/lint

Anyone who wants to add their own directories can do so by adding the
path to the 'include' directive at the bottom of:
tools/lint/flake8.lint

Of course, be sure there are no lint errors in this directory before
landing this change! You can run the linter locally with:
../mach lint -l flake8 <path>

The global flake8 configuration lives in topsrcdir/.flake8, but you can
override this global per subdirectory by adding a new .flake8 file in
that subdirectory. See [2] for configuration docs. We should definitely
bikeshed over which pep8 rules we wish to enforce/warn/ignore globally.
If you have strong feelings on any of them, please file a new thread for
that topic on mozilla.tools.

Finally, this task is using a new system called mozlint [3]. The aim of
mozlint is to standardize how we implement, run and report linters to
avoid re-inventing the wheel every time a new linter comes along. I'll
be switching the existing eslint implementation over to this as well. If
you have a linter you'd like to stand up locally or in automation,
please talk to me and I'll help point you in the right direction.

Let me know if you have any questions or spot any bugs!
Cheers,
Andrew

[1] https://gecko.readthedocs.io/en/latest/tools/lint/linter/flake8.html
[2] https://flake8.readthedocs.io/en/latest/config.html
[3] https://gecko.readthedocs.io/en/latest/tools/lint/index.html
_______________________________________________
dev-builds mailing list
dev-builds@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-builds

Reply via email to