diff options
author | Patrik Teivonen <patrik.teivonen@qt.io> | 2022-06-16 13:14:31 +0300 |
---|---|---|
committer | Patrik Teivonen <patrik.teivonen@qt.io> | 2022-07-11 09:07:49 +0000 |
commit | f1653086ad448000c46c01f7af8f742d1d5085a2 (patch) | |
tree | 7a7074463ec75a100a940abdcbc018755af4b9e3 /.pre-commit-config.yaml | |
parent | ee18c629245767ec020e6d78857b052f71569b2b (diff) |
PEP8: Fix anti-patterns
Enable following checks for anti-patterns in flake8:
- E502 The backslash is redundant between brackets
- E741 Do not use variables named 'I', 'O', or 'l'
- E722 Do not use bare except, specify exception instead
- W504 Line break occurred after a binary operator
Change-Id: I220159877e71a1380c54134c457546e437e43783
Reviewed-by: Akseli Salovaara <akseli.salovaara@qt.io>
Reviewed-by: Iikka Eklund <iikka.eklund@qt.io>
Diffstat (limited to '.pre-commit-config.yaml')
-rw-r--r-- | .pre-commit-config.yaml | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 217fe3ba9..cb5fc42a2 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -21,7 +21,7 @@ repos: name: Check PEP8 compliance (flake8) # Disable E203 for compatibility with blackformatter, and W503 as it goes against PEP8 # Disable checks that are not relevant to this patch, they will be introduced later - entry: bash -c 'pipenv run python3 -m flake8 --max-line-length=99 --ignore=E111,E117,E121,E123,E124,E125,E126,E127,E128,E201,E202,E203,E211,E221,E222,E225,E226,E231,E251,E252,E301,E302,E303,E305,E306,E501,E502,E722,E741,W391,W503,W504 "$@"' + entry: bash -c 'pipenv run python3 -m flake8 --max-line-length=99 --ignore=E111,E117,E121,E123,E124,E125,E126,E127,E128,E201,E202,E203,E211,E221,E222,E225,E226,E231,E251,E252,E301,E302,E303,E305,E306,E501,W391,W503 "$@"' language: system types: [python] fail_fast: true |