Skip to content

Commit

Permalink
Update flake8 to 7.1.1. (#327)
Browse files Browse the repository at this point in the history
We need to update flake8 to fix a false-positive that appears with older flake8 versions on Python 3.12.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - James Lamb (https://github.com/jameslamb)
  - Ben Frederickson (https://github.com/benfred)

URL: #327
  • Loading branch information
bdice authored Sep 17, 2024
1 parent e58b1d0 commit 09cd67c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ repos:
# Explicitly specify the pyproject.toml at the repo root, not per-project.
args: ["--config", "pyproject.toml"]
- repo: https://github.com/PyCQA/flake8
rev: 5.0.4
rev: 7.1.1
hooks:
- id: flake8
args: ["--config=.flake8"]
Expand Down
4 changes: 2 additions & 2 deletions python/cuvs/cuvs/test/test_doctests.py
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ def _find_doctests_in_obj(obj, finder=None, criteria=None):

def _test_name_from_docstring(docstring):
filename = Path(docstring.filename).name.split(".")[0]
return f"{filename}:{docstring.name}" # noqa: E231
return f"{filename}:{docstring.name}"


@pytest.mark.parametrize(
Expand All @@ -121,5 +121,5 @@ def test_docstring(docstring):
results = runner.summarize()
assert not results.failed, (
f"{results.failed} of {results.attempted} doctests failed for "
f"{docstring.name}:\n{doctest_stdout.getvalue()}" # noqa: E231
f"{docstring.name}:\n{doctest_stdout.getvalue()}"
)

0 comments on commit 09cd67c

Please sign in to comment.