similarity index 94%
rename from bindings/python/gpiod/internal.py
rename to bindings/python/gpiod/_internal.py
@@ -9,6 +9,7 @@ __all__ = ["poll_fd"]
def poll_fd(fd: int, timeout: Optional[Union[timedelta, float]] = None) -> bool:
+ sec: Union[float, None]
if isinstance(timeout, timedelta):
sec = timeout.total_seconds()
else:
@@ -7,10 +7,10 @@ from errno import ENOENT
from typing import Optional, Union
from . import _ext
+from ._internal import poll_fd
from .chip_info import ChipInfo
from .exception import ChipClosedError
from .info_event import InfoEvent
-from .internal import poll_fd
from .line import Value
from .line_info import LineInfo
from .line_request import LineRequest
@@ -6,9 +6,9 @@ from datetime import timedelta
from typing import Optional, Union
from . import _ext
+from ._internal import poll_fd
from .edge_event import EdgeEvent
from .exception import RequestReleasedError
-from .internal import poll_fd
from .line import Value
from .line_settings import LineSettings, _line_settings_to_ext
The internal submodule shouldn't be exposed as part of the public interface, so mark it private following PEP 8 convention [0]. Also, add a type annotation to `sec` for its possible types. [0]: https://peps.python.org/pep-0008/#public-and-internal-interfaces Signed-off-by: Vincent Fazio <vfazio@xes-inc.com> --- bindings/python/gpiod/{internal.py => _internal.py} | 1 + bindings/python/gpiod/chip.py | 2 +- bindings/python/gpiod/line_request.py | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-)