[Python] Fix code and format warnings (#4096)

- Format and sort __slots__ and __all__
- Replace soon to be deprecated datetime.utcnow
- Update ruff
- Better type annotations for the option module
- Use f-strings instead of format specifiers
- Fix dev dependency group in pyproject.toml

Author: dbrattli

poetry.lock

@@ -13,9 +13,9 @@ packages = [
 python = ">= 3.10, < 4.0"
 python-dateutil = "^2.9.0"
 
-[tool.poetry.dev-dependencies]
+[tool.poetry.group.dev.dependencies]
 pytest = "^8.1.1"
-ruff = "^0.3.4"
+ruff = "^0.11.6"
 
 [tool.pyright]
 reportMissingTypeStubs = false

pyproject.toml

@@ -35,12 +35,11 @@ def __init__(self, msg: str | None = None) -> None:
 
 
 class _Listener(Protocol):
-    def __call__(self, __state: Any | None = None) -> None:
-        ...
+    def __call__(self, __state: Any | None = None) -> None: ...
 
 
 class CancellationToken:
-    __slots__ = "cancelled", "listeners", "idx", "lock"
+    __slots__ = "cancelled", "idx", "listeners", "lock"
 
     def __init__(self, cancelled: bool = False):
         self.cancelled = cancelled
@@ -93,36 +92,29 @@ class IAsyncContext(Generic[_T]):
     __slots__ = ()
 
     @abstractmethod
-    def on_success(self, value: _T) -> None:
-        ...
+    def on_success(self, value: _T) -> None: ...
 
     @abstractmethod
-    def on_error(self, error: Exception) -> None:
-        ...
+    def on_error(self, error: Exception) -> None: ...
 
     @abstractmethod
-    def on_cancel(self, error: OperationCanceledError) -> None:
-        ...
+    def on_cancel(self, error: OperationCanceledError) -> None: ...
 
     @property
     @abstractmethod
-    def trampoline(self) -> Trampoline:
-        ...
+    def trampoline(self) -> Trampoline: ...
 
     @trampoline.setter
     @abstractmethod
-    def trampoline(self, val: Trampoline):
-        ...
+    def trampoline(self, val: Trampoline): ...
 
     @property
     @abstractmethod
-    def cancel_token(self) -> CancellationToken:
-        ...
+    def cancel_token(self) -> CancellationToken: ...
 
     @cancel_token.setter
     @abstractmethod
-    def cancel_token(self, val: CancellationToken):
-        ...
+    def cancel_token(self, val: CancellationToken): ...
 
     @staticmethod
     def create(
@@ -144,7 +136,7 @@ def empty_continuation(x: Any = None) -> None:
 
 
 class AnonymousAsyncContext(IAsyncContext[_T]):
-    __slots__ = "_on_success", "_on_error", "_on_cancel", "_trampoline", "_cancel_token"
+    __slots__ = "_cancel_token", "_on_cancel", "_on_error", "_on_success", "_trampoline"
 
     def __init__(
         self,
@@ -199,7 +191,7 @@ class ScheduledItem:
 
 
 class Trampoline:
-    __slots__ = "lock", "running", "call_count"
+    __slots__ = "call_count", "lock", "running"
 
     MaxTrampolineCallCount = 75  # Max recursion depth: 1000
 
@@ -308,12 +300,10 @@ def delay() -> Async[_U]:
         return self.While(lambda: not done, self.Delay(delay))
 
     @overload
-    def Return(self) -> Async[None]:
-        ...
+    def Return(self) -> Async[None]: ...
 
     @overload
-    def Return(self, value: _T) -> Async[_T]:
-        ...
+    def Return(self, value: _T) -> Async[_T]: ...
 
     def Return(self, value: Any = None) -> Async[Any]:
         return protected_return(value)
@@ -367,12 +357,10 @@ def compensation() -> None:
         return self.TryFinally(binder(resource), compensation)
 
     @overload
-    def While(self, guard: Callable[[], bool], computation: Async[Literal[None]]) -> Async[None]:
-        ...
+    def While(self, guard: Callable[[], bool], computation: Async[Literal[None]]) -> Async[None]: ...
 
     @overload
-    def While(self, guard: Callable[[], bool], computation: Async[_T]) -> Async[_T]:
-        ...
+    def While(self, guard: Callable[[], bool], computation: Async[_T]) -> Async[_T]: ...
 
     def While(self, guard: Callable[[], bool], computation: Async[Any]) -> Async[Any]:
         if guard():

src/fable-library-py/fable_library/async_builder.py

@@ -264,67 +264,67 @@ def try_parse(string: str, def_value: FSharpRef[int]) -> bool:
 
 __all__ = [
     "BigInteger",
+    "abs",
+    "add",
     "compare",
+    "divide",
     "equals",
-    "abs",
-    "sign",
+    "from_int32",
+    "from_int64",
+    "from_one",
+    "from_string",
+    "from_zero",
+    "get_is_even",
+    "get_is_one",
+    "get_is_power_of_two",
+    "get_is_zero",
+    "get_minus_one",
+    "get_one",
+    "get_sign",
+    "get_zero",
+    "is_even_integer",
+    "is_negative",
+    "is_odd_integer",
+    "is_positive",
+    "is_pow2",
     "max",
     "min",
-    "add",
-    "subtract",
     "multiply",
-    "divide",
-    "remainder",
     "negate",
-    "op_unary_negation",
-    "op_logical_not",
-    "op_unary_plus",
     "op_addition",
-    "op_subtraction",
-    "op_multiply",
-    "op_division",
-    "op_modulus",
-    "op_right_shift",
-    "op_left_shift",
     "op_bitwise_and",
     "op_bitwise_or",
+    "op_division",
+    "op_equality",
     "op_exclusive_or",
-    "op_less_than",
-    "op_less_than_or_equal",
     "op_greater_than",
     "op_greater_than_or_equal",
-    "op_equality",
     "op_inequality",
-    "get_zero",
-    "get_one",
-    "get_minus_one",
-    "get_is_zero",
-    "get_is_one",
-    "get_is_even",
-    "get_is_power_of_two",
-    "get_sign",
-    "is_negative",
-    "is_positive",
-    "is_even_integer",
-    "is_odd_integer",
-    "is_pow2",
-    "from_zero",
-    "from_one",
-    "from_int32",
-    "from_int64",
-    "from_string",
+    "op_left_shift",
+    "op_less_than",
+    "op_less_than_or_equal",
+    "op_logical_not",
+    "op_modulus",
+    "op_multiply",
+    "op_right_shift",
+    "op_subtraction",
+    "op_unary_negation",
+    "op_unary_plus",
+    "parse",
+    "remainder",
+    "sign",
+    "subtract",
+    "to_byte",
     "to_decimal",
     "to_double",
-    "to_single",
-    "to_uint64",
-    "to_int64",
-    "to_uint32",
-    "to_int32",
-    "to_uint16",
     "to_int16",
-    "to_byte",
+    "to_int32",
+    "to_int64",
     "to_sbyte",
+    "to_single",
     "to_string",
-    "parse",
+    "to_uint16",
+    "to_uint32",
+    "to_uint64",
     "try_parse",
 ]

src/fable-library-py/fable_library/big_int.py

@@ -106,26 +106,26 @@ def is_little_endian() -> bool:
 
 
 __all__ = [
+    "double_to_int64bits",
+    "get_bytes_boolean",
     "get_bytes_char",
+    "get_bytes_double",
     "get_bytes_int16",
-    "get_bytes_uint16",
     "get_bytes_int32",
-    "get_bytes_uint32",
     "get_bytes_int64",
-    "get_bytes_uint64",
-    "get_bytes_boolean",
     "get_bytes_single",
-    "get_bytes_double",
+    "get_bytes_uint16",
+    "get_bytes_uint32",
+    "get_bytes_uint64",
     "int64bits_to_double",
-    "double_to_int64bits",
+    "is_little_endian",
     "to_boolean",
     "to_char",
+    "to_double",
     "to_int16",
-    "to_uint16",
     "to_int32",
-    "to_uint64",
     "to_single",
-    "to_double",
     "to_string",
-    "is_little_endian",
+    "to_uint16",
+    "to_uint64",
 ]

src/fable-library-py/fable_library/bit_converter.py

@@ -230,20 +230,20 @@ def parse(input: str) -> str:
     "char_code_at",
     "get_unicode_category",
     "is_control",
-    "is_letter_or_digit",
-    "is_letter",
     "is_digit",
-    "is_upper",
+    "is_high_surrogate",
+    "is_letter",
+    "is_letter_or_digit",
+    "is_low_surrogate",
     "is_lower",
-    "is_separator",
-    "is_punctuation",
     "is_number",
-    "is_symbol",
+    "is_punctuation",
+    "is_separator",
     "is_separator",
-    "is_white_space",
     "is_surrogate",
-    "is_low_surrogate",
-    "is_high_surrogate",
     "is_surrogate_pair",
+    "is_symbol",
+    "is_upper",
+    "is_white_space",
     "parse",
 ]