MNT: flake8: fix E704, E226; ignore F824, W503, W504

Author: kernc

pdoc/test/__init__.py

@@ -289,7 +289,7 @@ def test_lunr_search(self):
                               file_pattern='example_pkg/subpkg/index.html')
         # Only build lunr search when --html
         with redirect_streams() as (_, stderr):
-            run(EXAMPLE_MODULE,  config='lunr_search={"fuzziness": 1}')
+            run(EXAMPLE_MODULE, config='lunr_search={"fuzziness": 1}')
             self.assertFalse(stderr.read())
 
     def test_force(self):
@@ -898,7 +898,7 @@ class Foo(enum.Enum):
         self.assertEqual(func.params(link=lambda x: ''), ['a=<object object>'])
 
         # typed
-        def f(a: int, *b, c: typing.List[pdoc.Doc] = []): pass
+        def f(a: int, *b, c: typing.List[pdoc.Doc] = []): pass  # noqa: E704
         func = pdoc.Function('f', mod, f)
         self.assertEqual(func.params(), ['a', '*b', "c=[]"])
         self.assertEqual(func.params(annotate=True),
@@ -913,14 +913,14 @@ def link(dobj):
                           "c:\N{NBSP}List[<a href=\"#pdoc.Doc\">Doc</a>]\N{NBSP}=\N{NBSP}[]"])
 
         # typed, linked, GH-311
-        def f(a: typing.Dict[str, pdoc.Doc]): pass
+        def f(a: typing.Dict[str, pdoc.Doc]): pass  # noqa: E704
 
         func = pdoc.Function('f', mod, f)
         self.assertEqual(func.params(annotate=True, link=link),
                          ["a:\N{NBSP}Dict[str,\N{NBSP}<a href=\"#pdoc.Doc\">Doc</a>]"])
 
         # shadowed name
-        def f(pdoc: int): pass
+        def f(pdoc: int): pass  # noqa: E704
         func = pdoc.Function('f', mod, f)
         self.assertEqual(func.params(annotate=True, link=link), ['pdoc:\N{NBSP}int'])
 
@@ -974,7 +974,7 @@ def test_test_Function_params_python38_specific(self):
         self.assertEqual(func.params(), ['a', '/'])
 
     def test_Function_return_annotation(self):
-        def f() -> typing.List[typing.Union[str, pdoc.Doc]]: return []
+        def f() -> typing.List[typing.Union[str, pdoc.Doc]]: return []  # noqa: E704
         func = pdoc.Function('f', DUMMY_PDOC_MODULE, f)
         self.assertEqual(func.return_annotation(), 'List[str\N{NBSP}|\N{NBSP}pdoc.Doc]')
 

pdoc/test/example_pkg/__init__.py

@@ -374,10 +374,10 @@ class Location(namedtuple('Location', 'lat lon')):
     """Geo-location, GPS position."""
 
 
-def _func_spec(value: int) -> bool: ...
+def _func_spec(value: int) -> bool: ...  # noqa: E704
 
 
-async def _coro_spec(value: int) -> bool: ...
+async def _coro_spec(value: int) -> bool: ...  # noqa: E704
 
 
 class HasMockAttributes:

pdoc/test/example_pkg/_skip_errors/unimportable.py

@@ -1 +1 @@
-1/0
+1 / 0

setup.cfg

@@ -1,5 +1,9 @@
 [flake8]
 max-line-length = 100
+# F824 `nonlocal x` is unused: name is never assigned in scope
+# W503 Line break before a binary operator
+# W504 Line break after a binary operator -- https://www.flake8rules.com/rules/W504.html
+ignore = F824, W503, W504
 
 [mypy]
 warn_unused_ignores = True