Update pre-commit dependencies

rra · rra · commit 3bc6a7df595c · 2024-11-25T13:10:35.000-08:00
Update pre-commit dependencies, apply fixes from Ruff 0.8.0, and
adjust the shared Ruff configuration file for some renamings of
diagnostic codes. Add a positional-only parameter separator in
one place detected by Ruff.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -8,14 +8,14 @@ repos:
       - id: trailing-whitespace
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.7.0
+    rev: v0.8.0
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]
       - id: ruff-format
 
   - repo: https://github.com/asottile/blacken-docs
-    rev: 1.19.0
+    rev: 1.19.1
     hooks:
       - id: blacken-docs
         additional_dependencies: [black==24.4.2]
diff --git a/ruff-shared.toml b/ruff-shared.toml
@@ -58,9 +58,9 @@ ignore = [
     "S607",     # using PATH is not a security vulnerability
     "SIM102",   # sometimes the formatting of nested if statements is clearer
     "SIM117",   # sometimes nested with contexts are clearer
-    "TCH001",   # we decided to not maintain separate TYPE_CHECKING blocks
-    "TCH002",   # we decided to not maintain separate TYPE_CHECKING blocks
-    "TCH003",   # we decided to not maintain separate TYPE_CHECKING blocks
+    "TC001",    # we decided to not maintain separate TYPE_CHECKING blocks
+    "TC002",    # we decided to not maintain separate TYPE_CHECKING blocks
+    "TC003",    # we decided to not maintain separate TYPE_CHECKING blocks
     "TD003",    # we don't require issues be created for TODOs
     "TID252",   # if we're going to use relative imports, use them always
     "TRY003",   # good general advice but lint is way too aggressive
diff --git a/safir-arq/src/safir/arq/__init__.py b/safir-arq/src/safir/arq/__init__.py
@@ -12,15 +12,15 @@
 
 __all__ = [
     "ArqJobError",
-    "JobNotQueued",
-    "JobNotFound",
-    "JobResultUnavailable",
     "ArqMode",
+    "ArqQueue",
     "JobMetadata",
+    "JobNotFound",
+    "JobNotQueued",
     "JobResult",
-    "ArqQueue",
-    "RedisArqQueue",
+    "JobResultUnavailable",
     "MockArqQueue",
-    "build_arq_redis_settings",
+    "RedisArqQueue",
     "WorkerSettings",
+    "build_arq_redis_settings",
 ]
diff --git a/safir-arq/src/safir/arq/_exceptions.py b/safir-arq/src/safir/arq/_exceptions.py
@@ -4,8 +4,8 @@
 
 __all__ = [
     "ArqJobError",
-    "JobNotQueued",
     "JobNotFound",
+    "JobNotQueued",
     "JobResultUnavailable",
 ]
 
diff --git a/safir-arq/src/safir/arq/_queue.py b/safir-arq/src/safir/arq/_queue.py
@@ -18,8 +18,8 @@
 
 __all__ = [
     "ArqQueue",
-    "RedisArqQueue",
     "MockArqQueue",
+    "RedisArqQueue",
 ]
 
 
diff --git a/safir-arq/src/safir/arq/uws.py b/safir-arq/src/safir/arq/uws.py
@@ -29,6 +29,8 @@
 """Name of the arq queue for internal UWS messages."""
 
 __all__ = [
+    "UWS_QUEUE_NAME",
+    "T",
     "WorkerConfig",
     "WorkerError",
     "WorkerErrorType",
@@ -38,8 +40,6 @@
     "WorkerTimeoutError",
     "WorkerTransientError",
     "WorkerUsageError",
-    "T",
-    "UWS_QUEUE_NAME",
     "build_worker",
 ]
 
diff --git a/safir-logging/src/safir/logging/__init__.py b/safir-logging/src/safir/logging/__init__.py
@@ -9,7 +9,7 @@
     "LogLevel",
     "Profile",
     "add_log_severity",
-    "configure_logging",
     "configure_alembic_logging",
+    "configure_logging",
     "configure_uvicorn_logging",
 ]
diff --git a/safir/src/safir/database/__init__.py b/safir/src/safir/database/__init__.py
@@ -29,10 +29,10 @@
     "AlembicConfigError",
     "DatabaseInitializationError",
     "DatetimeIdCursor",
-    "PaginationCursor",
     "PaginatedLinkData",
     "PaginatedList",
     "PaginatedQueryRunner",
+    "PaginationCursor",
     "create_async_session",
     "create_database_engine",
     "datetime_from_db",
diff --git a/safir/src/safir/database/_pagination.py b/safir/src/safir/database/_pagination.py
@@ -32,10 +32,10 @@
 
 __all__ = [
     "DatetimeIdCursor",
-    "PaginationCursor",
     "PaginatedLinkData",
     "PaginatedList",
     "PaginatedQueryRunner",
+    "PaginationCursor",
 ]
 
 
diff --git a/safir/src/safir/database/_retry.py b/safir/src/safir/database/_retry.py
@@ -28,7 +28,7 @@
 
 
 @overload
-def retry_async_transaction(__func: RetryF) -> RetryF: ...
+def retry_async_transaction(__func: RetryF, /) -> RetryF: ...
 
 
 @overload
@@ -38,7 +38,7 @@ def retry_async_transaction(
 
 
 def retry_async_transaction(
-    __func: RetryF | None = None, *, delay: float = 0.5, max_tries: int = 3
+    __func: RetryF | None = None, /, *, delay: float = 0.5, max_tries: int = 3
 ) -> RetryF | Callable[[RetryF], RetryF]:
     """Retry if a transaction failed.
 
diff --git a/safir/src/safir/dependencies/metrics.py b/safir/src/safir/dependencies/metrics.py
@@ -7,7 +7,7 @@
 
 from safir.metrics import EventManager
 
-__all__ = ["EventDependency", "EventMaker", "E"]
+__all__ = ["E", "EventDependency", "EventMaker"]
 
 E = TypeVar("E", bound="EventMaker")
 """Generic event maker type."""
diff --git a/safir/src/safir/github/models.py b/safir/src/safir/github/models.py
@@ -8,24 +8,24 @@
 from pydantic import BaseModel, Field, HttpUrl
 
 __all__ = [
-    "GitHubRepoOwnerModel",
-    "GitHubUserModel",
-    "GitHubRepositoryModel",
-    "GitHubPullState",
-    "GitHubPullRequestModel",
+    "GitHubBlobModel",
     "GitHubBranchCommitModel",
     "GitHubBranchModel",
-    "GitHubBlobModel",
-    "GitHubCheckSuiteStatus",
-    "GitHubCheckSuiteConclusion",
-    "GitHubCheckSuiteModel",
-    "GitHubCheckRunStatus",
-    "GitHubCheckRunConclusion",
     "GitHubCheckRunAnnotationLevel",
-    "GitHubCheckSuiteId",
+    "GitHubCheckRunConclusion",
+    "GitHubCheckRunModel",
     "GitHubCheckRunOutput",
     "GitHubCheckRunPrInfoModel",
-    "GitHubCheckRunModel",
+    "GitHubCheckRunStatus",
+    "GitHubCheckSuiteConclusion",
+    "GitHubCheckSuiteId",
+    "GitHubCheckSuiteModel",
+    "GitHubCheckSuiteStatus",
+    "GitHubPullRequestModel",
+    "GitHubPullState",
+    "GitHubRepoOwnerModel",
+    "GitHubRepositoryModel",
+    "GitHubUserModel",
 ]
 
 
diff --git a/safir/src/safir/github/webhooks.py b/safir/src/safir/github/webhooks.py
@@ -14,19 +14,19 @@
 )
 
 __all__ = [
-    "GitHubAppInstallationModel",
-    "GitHubPushEventModel",
-    "GitHubAppInstallationEventRepoModel",
     "GitHubAppInstallationEventAction",
     "GitHubAppInstallationEventModel",
+    "GitHubAppInstallationEventRepoModel",
+    "GitHubAppInstallationModel",
     "GitHubAppInstallationRepositoriesEventAction",
     "GitHubAppInstallationRepositoriesEventModel",
-    "GitHubPullRequestEventAction",
-    "GitHubPullRequestEventModel",
-    "GitHubCheckSuiteEventAction",
-    "GitHubCheckSuiteEventModel",
     "GitHubCheckRunEventAction",
     "GitHubCheckRunEventModel",
+    "GitHubCheckSuiteEventAction",
+    "GitHubCheckSuiteEventModel",
+    "GitHubPullRequestEventAction",
+    "GitHubPullRequestEventModel",
+    "GitHubPushEventModel",
 ]
 
 
diff --git a/safir/src/safir/kafka/__init__.py b/safir/src/safir/kafka/__init__.py
@@ -24,26 +24,26 @@
 
 __all__ = [
     "AIOKafkaParams",
+    "Compatibility",
     "FastStreamBrokerParams",
     "IncompatibleSchemaError",
     "InvalidAvroNameError",
     "InvalidMetadataError",
     "KafkaConnectionSettings",
     "PlaintextSettings",
     "PlaintextSettings",
+    "PydanticSchemaManager",
     "SaslMechanism",
     "SaslPlaintextSettings",
     "SaslSslSettings",
-    "SecurityProtocol",
-    "SslSettings",
-    "SslSettings",
-    "PydanticSchemaManager",
     "SchemaInfo",
     "SchemaManagerSettings",
     "SchemaManagerSettings",
     "SchemaRegistryClientParams",
     "SchemaRegistryClientParams",
-    "Compatibility",
+    "SecurityProtocol",
+    "SslSettings",
+    "SslSettings",
     "UnknownDeserializeError",
     "UnknownSchemaError",
 ]
diff --git a/safir/src/safir/kafka/_manager.py b/safir/src/safir/kafka/_manager.py
@@ -29,9 +29,9 @@
 )
 
 __all__ = [
+    "Compatibility",
     "PydanticSchemaManager",
     "SchemaInfo",
-    "Compatibility",
 ]
 
 P = TypeVar("P", bound=AvroBaseModel)
diff --git a/safir/src/safir/metrics/__init__.py b/safir/src/safir/metrics/__init__.py
@@ -25,12 +25,12 @@
     "BaseMetricsConfiguration",
     "DisabledMetricsConfiguration",
     "DuplicateEventError",
-    "EventsConfiguration",
     "EventManager",
     "EventManagerUnintializedError",
     "EventMetadata",
     "EventPayload",
     "EventPublisher",
+    "EventsConfiguration",
     "KafkaEventManager",
     "KafkaEventPublisher",
     "KafkaMetricsConfiguration",
diff --git a/safir/src/safir/metrics/_exceptions.py b/safir/src/safir/metrics/_exceptions.py
@@ -1,8 +1,8 @@
 """Metrics exceptions."""
 
 __all__ = [
-    "EventManagerUnintializedError",
     "DuplicateEventError",
+    "EventManagerUnintializedError",
     "KafkaTopicError",
 ]
 
diff --git a/safir/src/safir/redis/_storage.py b/safir/src/safir/redis/_storage.py
@@ -23,9 +23,9 @@
 )
 
 __all__ = [
-    "PydanticRedisStorage",
-    "EncryptedPydanticRedisStorage",
     "DeserializeError",
+    "EncryptedPydanticRedisStorage",
+    "PydanticRedisStorage",
 ]
 
 
diff --git a/safir/src/safir/uws/__init__.py b/safir/src/safir/uws/__init__.py
@@ -27,8 +27,8 @@
     "ParameterError",
     "ParameterParseError",
     "ParametersModel",
-    "UWSApplication",
     "UWSAppSettings",
+    "UWSApplication",
     "UWSConfig",
     "UWSError",
     "UWSJob",
diff --git a/safir/src/safir/uws/_constants.py b/safir/src/safir/uws/_constants.py
@@ -7,8 +7,8 @@
 from arq.cron import Options
 
 __all__ = [
-    "JOB_STOP_TIMEOUT",
     "JOB_RESULT_TIMEOUT",
+    "JOB_STOP_TIMEOUT",
     "UWS_DATABASE_TIMEOUT",
     "UWS_EXPIRE_JOBS_SCHEDULE",
 ]
diff --git a/safir/src/safir/uws/_exceptions.py b/safir/src/safir/uws/_exceptions.py
@@ -32,9 +32,9 @@
     "SyncJobNoResultsError",
     "SyncJobTimeoutError",
     "TaskError",
+    "UWSError",
     "UnknownJobError",
     "UsageError",
-    "UWSError",
 ]
 
 

Original file line number	Diff line number	Diff line change
`@@ -4,8 +4,8 @@`
`4`	`4`
`5`	`5`	`__all__ = [`
`6`	`6`	`"ArqJobError",`
`7`		`- "JobNotQueued",`
`8`	`7`	`"JobNotFound",`
	`8`	`+ "JobNotQueued",`
`9`	`9`	`"JobResultUnavailable",`
`10`	`10`	`]`
`11`	`11`
Original file line number	Diff line number	Diff line change
`@@ -18,8 +18,8 @@`
`18`	`18`
`19`	`19`	`__all__ = [`
`20`	`20`	`"ArqQueue",`
`21`		`- "RedisArqQueue",`
`22`	`21`	`"MockArqQueue",`
	`22`	`+ "RedisArqQueue",`
`23`	`23`	`]`
`24`	`24`
`25`	`25`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`"LogLevel",`
`10`	`10`	`"Profile",`
`11`	`11`	`"add_log_severity",`
`12`		`- "configure_logging",`
`13`	`12`	`"configure_alembic_logging",`
	`13`	`+ "configure_logging",`
`14`	`14`	`"configure_uvicorn_logging",`
`15`	`15`	`]`
Original file line number	Diff line number	Diff line change
`@@ -32,10 +32,10 @@`
`32`	`32`
`33`	`33`	`__all__ = [`
`34`	`34`	`"DatetimeIdCursor",`
`35`		`- "PaginationCursor",`
`36`	`35`	`"PaginatedLinkData",`
`37`	`36`	`"PaginatedList",`
`38`	`37`	`"PaginatedQueryRunner",`
	`38`	`+ "PaginationCursor",`
`39`	`39`	`]`
`40`	`40`
`41`	`41`
Original file line number	Diff line number	Diff line change
`@@ -29,9 +29,9 @@`
`29`	`29`	`)`
`30`	`30`
`31`	`31`	`__all__ = [`
	`32`	`+ "Compatibility",`
`32`	`33`	`"PydanticSchemaManager",`
`33`	`34`	`"SchemaInfo",`
`34`		`- "Compatibility",`
`35`	`35`	`]`
`36`	`36`
`37`	`37`	`P = TypeVar("P", bound=AvroBaseModel)`
Original file line number	Diff line number	Diff line change
`@@ -1,8 +1,8 @@`
`1`	`1`	`"""Metrics exceptions."""`
`2`	`2`
`3`	`3`	`__all__ = [`
`4`		`- "EventManagerUnintializedError",`
`5`	`4`	`"DuplicateEventError",`
	`5`	`+ "EventManagerUnintializedError",`
`6`	`6`	`"KafkaTopicError",`
`7`	`7`	`]`
`8`	`8`
Original file line number	Diff line number	Diff line change
`@@ -23,9 +23,9 @@`
`23`	`23`	`)`
`24`	`24`
`25`	`25`	`__all__ = [`
`26`		`- "PydanticRedisStorage",`
`27`		`- "EncryptedPydanticRedisStorage",`
`28`	`26`	`"DeserializeError",`
	`27`	`+ "EncryptedPydanticRedisStorage",`
	`28`	`+ "PydanticRedisStorage",`
`29`	`29`	`]`
`30`	`30`
`31`	`31`
Original file line number	Diff line number	Diff line change
`@@ -7,8 +7,8 @@`
`7`	`7`	`from arq.cron import Options`
`8`	`8`
`9`	`9`	`__all__ = [`
`10`		`- "JOB_STOP_TIMEOUT",`
`11`	`10`	`"JOB_RESULT_TIMEOUT",`
	`11`	`+ "JOB_STOP_TIMEOUT",`
`12`	`12`	`"UWS_DATABASE_TIMEOUT",`
`13`	`13`	`"UWS_EXPIRE_JOBS_SCHEDULE",`
`14`	`14`	`]`
Original file line number	Diff line number	Diff line change
`@@ -32,9 +32,9 @@`
`32`	`32`	`"SyncJobNoResultsError",`
`33`	`33`	`"SyncJobTimeoutError",`
`34`	`34`	`"TaskError",`
	`35`	`+ "UWSError",`
`35`	`36`	`"UnknownJobError",`
`36`	`37`	`"UsageError",`
`37`		`- "UWSError",`
`38`	`38`	`]`
`39`	`39`
`40`	`40`