Reduce usage of # noqa.

Specify which warnings are ignored.

src/websockets/__init__.py

@@ -1,10 +1,10 @@
 from __future__ import annotations
 
 from .imports import lazy_import
-from .version import version as __version__  # noqa
+from .version import version as __version__  # noqa: F401
 
 
-__all__ = [  # noqa
+__all__ = [
     "AbortHandshake",
     "basic_auth_protocol_factory",
     "BasicAuthWebSocketServerProtocol",

src/websockets/__main__.py

@@ -8,7 +8,7 @@
 
 
 try:
-    import readline  # noqa
+    import readline  # noqa: F401
 except ImportError:  # Windows has no `readline` normally
     pass
 

src/websockets/auth.py

@@ -1,4 +1,4 @@
 from __future__ import annotations
 
 # See #940 for why lazy_import isn't used here for backwards compatibility.
-from .legacy.auth import *  # noqa
+from .legacy.auth import *

src/websockets/client.py

@@ -38,7 +38,7 @@
 
 
 # See #940 for why lazy_import isn't used here for backwards compatibility.
-from .legacy.client import *  # isort:skip  # noqa
+from .legacy.client import *  # isort:skip  # noqa: I001
 
 
 __all__ = ["ClientProtocol"]
@@ -90,7 +90,7 @@ def __init__(
         self.available_subprotocols = subprotocols
         self.key = generate_key()
 
-    def connect(self) -> Request:  # noqa: F811
+    def connect(self) -> Request:
         """
         Create a handshake request to open a connection.
 

src/websockets/connection.py

@@ -3,7 +3,7 @@
 import warnings
 
 # lazy_import doesn't support this use case.
-from .protocol import SEND_EOF, Protocol as Connection, Side, State  # noqa
+from .protocol import SEND_EOF, Protocol as Connection, Side, State  # noqa: F401
 
 
 warnings.warn(

src/websockets/legacy/framing.py

@@ -145,8 +145,11 @@ def write(
 
 
 # Backwards compatibility with previously documented public APIs
-
-from ..frames import Close, prepare_ctrl as encode_data, prepare_data  # noqa
+from ..frames import (  # noqa: E402, F401, I001
+    Close,
+    prepare_ctrl as encode_data,
+    prepare_data,
+)
 
 
 def parse_close(data: bytes) -> Tuple[int, str]:

src/websockets/server.py

@@ -39,7 +39,7 @@
 
 
 # See #940 for why lazy_import isn't used here for backwards compatibility.
-from .legacy.server import *  # isort:skip  # noqa
+from .legacy.server import *  # isort:skip  # noqa: I001
 
 
 __all__ = ["ServerProtocol"]

tests/extensions/test_base.py

@@ -1,4 +1,4 @@
-from websockets.extensions.base import *  # noqa
+from websockets.extensions.base import *
 
 
 # Abstract classes don't provide any behavior to test.

tests/test_auth.py

@@ -1 +1 @@
-from websockets.auth import *  # noqa
+from websockets.auth import *

tests/test_http.py

@@ -1 +1 @@
-from websockets.http import *  # noqa
+from websockets.http import *

tests/test_typing.py

@@ -1 +1 @@
-from websockets.typing import *  # noqa
+from websockets.typing import *