Skip to content

Commit

Permalink
🩹 Fix yesqa by moving count from the config to the hook args
Browse files Browse the repository at this point in the history
When using count yesqa crashes due to tuple unpacking when splitting on '\t'
  • Loading branch information
s-weigand committed Nov 7, 2022
1 parent f1560a3 commit ec502da
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
3 changes: 3 additions & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,8 @@ repos:
types_or: [python, pyi]
additional_dependencies:
[flake8-pyi, flake8-comprehensions, flake8-print>=5.0.0]
args:
- "--count"

- repo: https://github.com/PyCQA/flake8
rev: 5.0.4
Expand All @@ -105,6 +107,7 @@ repos:
alias: flake8-docs
args:
- "--select=D,DAR"
- "--count"
name: "flake8 lint docstrings"
exclude: "docs|setup.py|tests?/|glotaran/builtin/megacomplexes|glotaran/cli|benchmark|glotaran/builtin/io/ascii"
additional_dependencies: [flake8-docstrings, darglint==1.8.0]
Expand Down
1 change: 0 additions & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ envlist = py{38,39,310}, pre-commit, docs, docs-notebooks, docs-links


[flake8]
count = True
; D414 is needed due to a bug in pydocstyle the sees the returntype `Parameters` as section
extend-ignore = E231, E203, D414
max-line-length = 99
Expand Down

0 comments on commit ec502da

Please sign in to comment.