Xqt has submitted this change. ( https://gerrit.wikimedia.org/r/c/pywikibot/core/+/1132171?usp=email )

Change subject: tests: update pre-commit hooks ......................................................................

tests: update pre-commit hooks

Ignore F824 which gives error messages in archivebot and unidata but they looks wrong.

Anyway we should avoid globals.

Change-Id: Icc1cff10ed2c7928e242c4e2e69a6d54f3035177 --- M .pre-commit-config.yaml M tox.ini 2 files changed, 3 insertions(+), 2 deletions(-)

Approvals: Xqt: Verified; Looks good to me, approved

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 51e62ee..667741c 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -91,7 +91,7 @@ - id: docsig exclude: ^(tests|scripts) - repo: https://github.com/PyCQA/flake8 - rev: 7.1.2 + rev: 7.2.0 hooks: - id: flake8 args: diff --git a/tox.ini b/tox.ini index 4d474e9..9585eed 100644 --- a/tox.ini +++ b/tox.ini @@ -120,10 +120,11 @@ # B007: Loop control variable not used within the loop body # B028: No explicit stacklevel argument found # E704: multiple statements on one line (def) +# F824: False positives in flake8 7.2.0 # R100: raise in except handler without from # W503: line break before binary operator; against current PEP 8 recommendation

-ignore = B007,B028,E704,R100,W503 +ignore = B007,B028,E704,F824,R100,W503 enable-extensions = N818

count = True