Skip to content

Commit

Permalink
Bug 1806068 - Ignore formatting issue in flake8 r=linter-reviewers,sy…
Browse files Browse the repository at this point in the history
…lvestre DONTBUILD

They are redundant with black formatting, so just ignore them.

Differential Revision: https://phabricator.services.mozilla.com/D164884
  • Loading branch information
serge-sans-paille committed Dec 18, 2022
1 parent 4273a3f commit 3d4d823
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions .flake8
Original file line number Diff line number Diff line change
Expand Up @@ -79,15 +79,15 @@ exclude =
# - http://pep8.readthedocs.io/en/latest/intro.html#configuration
ignore =
# These should be triaged and either fixed or moved to the list below.
W504, W605, W606,
W605, W606,
# These are intentionally disabled (not necessarily for good reason).
# F723: syntax error in type comment
# text contains quotes which breaks our custom JSON formatter
F723, E121, E123, E126, E129, E133, E226, E241, E242, E402, E704, E741, W503,
F723, E704, E741,

# black will generate code that breaks these. they're not PEP8 compliant though;
# see https://github.com/psf/black/blob/master/docs/compatible_configs.md#flake8
W503, E203
# black is already in charge of formatting, no need to start a formatter
# battle here
E1, W1, E2, W2, E3, W3, E4, W4, E5, W5

per-file-ignores =
# These paths are intentionally excluded.
Expand Down

0 comments on commit 3d4d823

Please sign in to comment.