Update Ruff

hynek · Dec 3, 2024 · a60ce7b · a60ce7b
1 parent 6a111d6
commit a60ce7b
Show file tree

Hide file tree

Showing 8 changed files with 27 additions and 26 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -4,7 +4,7 @@ ci:
 
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.7.3
+    rev: v0.8.1
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]

diff --git a/src/structlog/__init__.py b/src/structlog/__init__.py
@@ -61,23 +61,25 @@
     "BoundLoggerBase",
     "BytesLogger",
     "BytesLoggerFactory",
-    "configure_once",
+    "DropEvent",
+    "PrintLogger",
+    "PrintLoggerFactory",
+    "ReturnLogger",
+    "ReturnLoggerFactory",
+    "WriteLogger",
+    "WriteLoggerFactory",
     "configure",
+    "configure_once",
     "contextvars",
     "dev",
-    "DropEvent",
+    "getLogger",
     "get_config",
     "get_context",
     "get_logger",
-    "getLogger",
     "is_configured",
     "make_filtering_bound_logger",
-    "PrintLogger",
-    "PrintLoggerFactory",
     "processors",
     "reset_defaults",
-    "ReturnLogger",
-    "ReturnLoggerFactory",
     "stdlib",
     "testing",
     "threadlocal",
@@ -86,8 +88,6 @@
     "types",
     "typing",
     "wrap_logger",
-    "WriteLogger",
-    "WriteLoggerFactory",
 ]
 
 

diff --git a/src/structlog/_output.py b/src/structlog/_output.py
@@ -256,7 +256,7 @@ class BytesLogger:
     .. versionadded:: 20.2.0
     """
 
-    __slots__ = ("_file", "_write", "_flush", "_lock")
+    __slots__ = ("_file", "_flush", "_lock", "_write")
 
     def __init__(self, file: BinaryIO | None = None):
         self._file = file or sys.stdout.buffer

diff --git a/src/structlog/dev.py b/src/structlog/dev.py
@@ -55,9 +55,9 @@
 
 __all__ = [
     "ConsoleRenderer",
+    "better_traceback",
     "plain_traceback",
     "rich_traceback",
-    "better_traceback",
 ]
 
 _IS_WINDOWS = sys.platform == "win32"

diff --git a/src/structlog/processors.py b/src/structlog/processors.py
@@ -43,19 +43,19 @@
 
 __all__ = [
     "NAME_TO_LEVEL",  # some people rely on it being here
-    "add_log_level",
     "CallsiteParameter",
     "CallsiteParameterAdder",
-    "dict_tracebacks",
     "EventRenamer",
     "ExceptionPrettyPrinter",
-    "format_exc_info",
     "JSONRenderer",
     "KeyValueRenderer",
     "StackInfoRenderer",
     "TimeStamper",
     "UnicodeDecoder",
     "UnicodeEncoder",
+    "add_log_level",
+    "dict_tracebacks",
+    "format_exc_info",
 ]
 
 
@@ -472,7 +472,7 @@ class TimeStamper:
     .. versionchanged:: 19.2.0 Can be pickled now.
     """
 
-    __slots__ = ("_stamper", "fmt", "utc", "key")
+    __slots__ = ("_stamper", "fmt", "key", "utc")
 
     def __init__(
         self,

diff --git a/src/structlog/stdlib.py b/src/structlog/stdlib.py
@@ -39,16 +39,16 @@
 
 
 __all__ = [
-    "add_log_level_number",
-    "add_log_level",
-    "add_logger_name",
-    "ExtraAdder",
     "BoundLogger",
-    "filter_by_level",
-    "get_logger",
+    "ExtraAdder",
     "LoggerFactory",
     "PositionalArgumentsFormatter",
     "ProcessorFormatter",
+    "add_log_level",
+    "add_log_level_number",
+    "add_logger_name",
+    "filter_by_level",
+    "get_logger",
     "recreate_defaults",
     "render_to_log_kwargs",
 ]
@@ -521,7 +521,7 @@ class AsyncBoundLogger:
         Callsite parameters are now also collected for async log methods.
     """
 
-    __slots__ = ("sync_bl", "_loop")
+    __slots__ = ("_loop", "sync_bl")
 
     #: The wrapped synchronous logger. It is useful to be able to log
     #: synchronously occasionally.

diff --git a/src/structlog/types.py b/src/structlog/types.py
@@ -26,13 +26,13 @@
 
 
 __all__ = (
-    "WrappedLogger",
+    "BindableLogger",
     "Context",
     "EventDict",
-    "Processor",
     "ExcInfo",
     "ExceptionRenderer",
     "ExceptionTransformer",
-    "BindableLogger",
     "FilteringBoundLogger",
+    "Processor",
+    "WrappedLogger",
 )
diff --git a/tests/typing/api.py b/tests/typing/api.py
@@ -29,6 +29,7 @@
 
 def bytes_dumps(
     __obj: Any,
+    /,
     default: Callable[[Any], Any] | None = None,
     option: int | None = None,
 ) -> bytes: