Export symbols to satisfy type checkers (#1551)

* use ruff with isort, export symbols according to PEP484

* fix CI

* fix CI

* use __all__

Author: zariiii9003

.github/workflows/ci.yml

@@ -95,6 +95,9 @@ jobs:
     - name: mypy 3.11
       run: |
         mypy --python-version 3.11 .
+    - name: ruff
+      run: |
+        ruff check can
     - name: pylint
       run: |
         pylint --rcfile=.pylintrc \

.pylintrc

@@ -438,6 +438,8 @@ known-standard-library=
 # Force import order to recognize a module as part of a third party library.
 known-third-party=enchant
 
+# Allow explicit reexports by alias from a package __init__
+allow-reexport-from-package=no
 
 [CLASSES]
 

can/__init__.py

@@ -6,47 +6,104 @@
 """
 
 import logging
-from typing import Dict, Any
+from typing import Any, Dict
 
 __version__ = "4.1.0"
+__all__ = [
+    "ASCReader",
+    "ASCWriter",
+    "AsyncBufferedReader",
+    "BitTiming",
+    "BitTimingFd",
+    "BLFReader",
+    "BLFWriter",
+    "broadcastmanager",
+    "BufferedReader",
+    "Bus",
+    "BusABC",
+    "BusState",
+    "CanError",
+    "CanInitializationError",
+    "CanInterfaceNotImplementedError",
+    "CanOperationError",
+    "CanTimeoutError",
+    "CanutilsLogReader",
+    "CanutilsLogWriter",
+    "CSVReader",
+    "CSVWriter",
+    "CyclicSendTaskABC",
+    "detect_available_configs",
+    "interface",
+    "LimitedDurationCyclicSendTaskABC",
+    "Listener",
+    "Logger",
+    "LogReader",
+    "ModifiableCyclicTaskABC",
+    "Message",
+    "MessageSync",
+    "Notifier",
+    "Printer",
+    "RedirectReader",
+    "RestartableCyclicTaskABC",
+    "set_logging_level",
+    "SizedRotatingLogger",
+    "SqliteReader",
+    "SqliteWriter",
+    "ThreadSafeBus",
+    "typechecking",
+    "TRCFileVersion",
+    "TRCReader",
+    "TRCWriter",
+    "util",
+    "VALID_INTERFACES",
+]
 
 log = logging.getLogger("can")
 
 rc: Dict[str, Any] = {}
 
-from .listener import Listener, BufferedReader, RedirectReader, AsyncBufferedReader
-
+from . import typechecking  # isort:skip
+from . import util  # isort:skip
+from . import broadcastmanager, interface
+from .bit_timing import BitTiming, BitTimingFd
+from .broadcastmanager import (
+    CyclicSendTaskABC,
+    LimitedDurationCyclicSendTaskABC,
+    ModifiableCyclicTaskABC,
+    RestartableCyclicTaskABC,
+)
+from .bus import BusABC, BusState
 from .exceptions import (
     CanError,
-    CanInterfaceNotImplementedError,
     CanInitializationError,
+    CanInterfaceNotImplementedError,
     CanOperationError,
     CanTimeoutError,
 )
-
-from .util import set_logging_level
-
-from .message import Message
-from .bus import BusABC, BusState
-from .thread_safe_bus import ThreadSafeBus
-from .notifier import Notifier
-from .interfaces import VALID_INTERFACES
-from . import interface
 from .interface import Bus, detect_available_configs
-from .bit_timing import BitTiming, BitTimingFd
-
-from .io import Logger, SizedRotatingLogger, Printer, LogReader, MessageSync
-from .io import ASCWriter, ASCReader
-from .io import BLFReader, BLFWriter
-from .io import CanutilsLogReader, CanutilsLogWriter
-from .io import CSVWriter, CSVReader
-from .io import SqliteWriter, SqliteReader
-from .io import TRCReader, TRCWriter, TRCFileVersion
-
-from .broadcastmanager import (
-    CyclicSendTaskABC,
-    LimitedDurationCyclicSendTaskABC,
-    ModifiableCyclicTaskABC,
-    MultiRateCyclicSendTaskABC,
-    RestartableCyclicTaskABC,
+from .interfaces import VALID_INTERFACES
+from .io import (
+    ASCReader,
+    ASCWriter,
+    BLFReader,
+    BLFWriter,
+    CanutilsLogReader,
+    CanutilsLogWriter,
+    CSVReader,
+    CSVWriter,
+    Logger,
+    LogReader,
+    MessageSync,
+    Printer,
+    SizedRotatingLogger,
+    SqliteReader,
+    SqliteWriter,
+    TRCFileVersion,
+    TRCReader,
+    TRCWriter,
 )
+from .listener import AsyncBufferedReader, BufferedReader, Listener, RedirectReader
+from .message import Message
+from .notifier import Notifier
+from .thread_safe_bus import ThreadSafeBus
+from .util import set_logging_level

can/bit_timing.py

@@ -1,8 +1,8 @@
 # pylint: disable=too-many-lines
 import math
-from typing import List, Mapping, Iterator, cast
+from typing import Iterator, List, Mapping, cast
 
-from can.typechecking import BitTimingFdDict, BitTimingDict
+from can.typechecking import BitTimingDict, BitTimingFdDict
 
 
 class BitTiming(Mapping):

can/broadcastmanager.py

@@ -10,7 +10,7 @@
 import sys
 import threading
 import time
-from typing import Optional, Sequence, Tuple, Union, Callable, TYPE_CHECKING
+from typing import TYPE_CHECKING, Callable, Optional, Sequence, Tuple, Union
 
 from typing_extensions import Final
 

can/bus.py

@@ -1,19 +1,18 @@
 """
 Contains the ABC bus implementation and its documentation.
 """
-import contextlib
-from typing import cast, Any, Iterator, List, Optional, Sequence, Tuple, Union
-
-import can.typechecking
 
-from abc import ABC, ABCMeta, abstractmethod
-import can
+import contextlib
 import logging
 import threading
-from time import time
+from abc import ABC, ABCMeta, abstractmethod
 from enum import Enum, auto
+from time import time
+from typing import Any, Iterator, List, Optional, Sequence, Tuple, Union, cast
 
-from can.broadcastmanager import ThreadBasedCyclicSendTask, CyclicSendTaskABC
+import can
+import can.typechecking
+from can.broadcastmanager import CyclicSendTaskABC, ThreadBasedCyclicSendTask
 from can.message import Message
 
 LOG = logging.getLogger(__name__)

can/ctypesutil.py

@@ -5,7 +5,6 @@
 import ctypes
 import logging
 import sys
-
 from typing import Any, Callable, Optional, Tuple, Union
 
 log = logging.getLogger("can.ctypesutil")

can/exceptions.py

@@ -17,9 +17,7 @@
 
 import sys
 from contextlib import contextmanager
-
-from typing import Optional
-from typing import Type
+from typing import Optional, Type
 
 if sys.version_info >= (3, 9):
     from collections.abc import Generator

can/interface.py

@@ -6,12 +6,12 @@
 
 import importlib
 import logging
-from typing import Any, cast, Iterable, Type, Optional, Union, List
+from typing import Any, Iterable, List, Optional, Type, Union, cast
 
 from . import util
 from .bus import BusABC
-from .interfaces import BACKENDS
 from .exceptions import CanInterfaceNotImplementedError
+from .interfaces import BACKENDS
 from .typechecking import AutoDetectedConfig, Channel
 
 log = logging.getLogger("can.interface")

can/interfaces/__init__.py

@@ -3,7 +3,7 @@
 """
 
 import sys
-from typing import cast, Dict, Tuple
+from typing import Dict, Tuple, cast
 
 # interface_name => (module, classname)
 BACKENDS: Dict[str, Tuple[str, str]] = {