Modernize code style by linters

.github/dependabot.yml

@@ -5,3 +5,5 @@ updates:
     schedule:
       interval: "weekly"
     target-branch: develop
+    cooldown:
+      default-days: 28

.github/workflows/lint_and_test.yml

@@ -28,7 +28,7 @@ jobs:
           persist-credentials: false
 
       - name: Set up Python 3.11
-        uses: actions/setup-python@v6
+        uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405
         with:
           python-version: "3.11"
 

.pre-commit-config.yaml

@@ -18,7 +18,7 @@ default_stages:
 
 repos:
   - repo: 'https://github.com/pre-commit/pre-commit-hooks'
-    rev: v5.0.0
+    rev: v6.0.0
     hooks:
       - id: check-merge-conflict
       - id: debug-statements
@@ -35,24 +35,24 @@ repos:
         args: ['-e']  # remove default --in-place argument and emit error
 
   - repo: 'https://github.com/astral-sh/ruff-pre-commit'
-    rev: v0.14.0
+    rev: v0.14.14
     hooks:
       - id: ruff
         args: ['--fix-only']
 
   - repo: 'https://github.com/python-jsonschema/check-jsonschema'
-    rev: 0.33.0
+    rev: 0.36.1
     hooks:
       - id: check-dependabot
       - id: check-github-workflows
 
   - repo: 'https://github.com/rhysd/actionlint'
-    rev: v1.7.7
+    rev: v1.7.10
     hooks:
       - id: actionlint
 
   - repo: 'https://github.com/woodruffw/zizmor-pre-commit'
-    rev: v1.8.0
+    rev: v1.22.0
     hooks:
       - id: zizmor
 

benchmarks/benchmarks/bench_nexus.py

@@ -12,7 +12,7 @@
 from itertools import pairwise
 from pathlib import Path
 from textwrap import dedent
-from typing import Any, Callable, DefaultDict, Dict, Iterable, List, Mapping, Optional, Sequence, Set, TypeVar, Union
+from typing import Any, Callable, DefaultDict, Dict, Iterable, List, Mapping, Sequence, Set, TypeVar
 from zipfile import ZIP_BZIP2, ZipFile
 
 import plotly
@@ -39,7 +39,7 @@ class BenchmarkMeasure:
     pyperf: pyperf.Benchmark
 
 
-def pyperf_bench_to_measure(data: Union[str, bytes]) -> BenchmarkMeasure:
+def pyperf_bench_to_measure(data: str | bytes) -> BenchmarkMeasure:
     pybench_data = json.loads(data)["pybench_data"]
     return BenchmarkMeasure(
         base=pybench_data["base"],
@@ -243,10 +243,10 @@ def add_arguments(cls, parser: ArgumentParser) -> None:
     def __init__(
         self,
         namespace: Namespace,
-        include: Optional[Sequence[str]] = None,
-        exclude: Optional[Sequence[str]] = None,
-        env_include: Optional[Sequence[str]] = None,
-        env_exclude: Optional[Sequence[str]] = None,
+        include: Sequence[str] | None = None,
+        exclude: Sequence[str] | None = None,
+        env_include: Sequence[str] | None = None,
+        env_exclude: Sequence[str] | None = None,
     ):
         self.include = include
         self.exclude = exclude
@@ -328,7 +328,7 @@ def load_directors(
                     env_description: self._submit_python(executor, env_description, env_spec_printer)
                     for env_description in self.filtered_envs()
                 }
-                for hub_description, env_spec_printer in zip(hub_descriptions, env_spec_printers)
+                for hub_description, env_spec_printer in zip(hub_descriptions, env_spec_printers, strict=False)
             }
             return {
                 hub_description: {
@@ -354,7 +354,7 @@ class CaseState:
     accessor: BenchAccessor
     checker: BenchChecker
     local_ids_with_warnings: Sequence[str]
-    max_tries: Optional[int]
+    max_tries: int | None
     tries_count: int = 0
 
     @property
@@ -388,12 +388,12 @@ def add_arguments(cls, parser: ArgumentParser) -> None:
     def __init__(
         self,
         namespace: Namespace,
-        include: Optional[Sequence[str]] = None,
-        exclude: Optional[Sequence[str]] = None,
-        env_include: Optional[Sequence[str]] = None,
-        env_exclude: Optional[Sequence[str]] = None,
+        include: Sequence[str] | None = None,
+        exclude: Sequence[str] | None = None,
+        env_include: Sequence[str] | None = None,
+        env_exclude: Sequence[str] | None = None,
         series: int = 2,
-        max_tries: Optional[int] = None,
+        max_tries: int | None = None,
     ):
         super().__init__(
             namespace=namespace,
@@ -551,11 +551,11 @@ def add_arguments(cls, parser: ArgumentParser) -> None:
     def __init__(
         self,
         namespace: Namespace,
-        include: Optional[Sequence[str]] = None,
-        exclude: Optional[Sequence[str]] = None,
-        env_include: Optional[Sequence[str]] = None,
-        env_exclude: Optional[Sequence[str]] = None,
-        output: Optional[str] = None,
+        include: Sequence[str] | None = None,
+        exclude: Sequence[str] | None = None,
+        env_include: Sequence[str] | None = None,
+        env_exclude: Sequence[str] | None = None,
+        output: str | None = None,
     ):
         super().__init__(
             namespace=namespace,

benchmarks/benchmarks/gh_issues/bench_adaptix.py

@@ -1,6 +1,6 @@
 from dataclasses import dataclass
 from datetime import datetime
-from typing import List, Optional
+from typing import List
 
 from adaptix import DebugTrail, Retort, name_mapping
 from benchmarks.gh_issues.common import (
@@ -19,7 +19,7 @@ class SimpleUser:
     id: int
     node_id: str
     avatar_url: str
-    gravatar_id: Optional[str]
+    gravatar_id: str | None
     url: str
     html_url: str
     followers_url: str
@@ -33,9 +33,9 @@ class SimpleUser:
     received_events_url: str
     type: str
     site_admin: bool
-    name: Optional[str] = None
-    email: Optional[str] = None
-    starred_at: Optional[datetime] = None
+    name: str | None = None
+    email: str | None = None
+    starred_at: datetime | None = None
 
 
 @dataclass
@@ -44,7 +44,7 @@ class Label:
     node_id: str
     url: str
     name: str
-    description: Optional[str]
+    description: str | None
     color: str
     default: bool
 
@@ -65,11 +65,11 @@ class Reactions:
 
 @dataclass
 class PullRequest:
-    diff_url: Optional[str]
-    html_url: Optional[str]
-    patch_url: Optional[str]
-    url: Optional[str]
-    merged_at: Optional[datetime] = None
+    diff_url: str | None
+    html_url: str | None
+    patch_url: str | None
+    url: str | None
+    merged_at: datetime | None = None
 
 
 @dataclass
@@ -84,25 +84,25 @@ class Issue:
     html_url: str
     number: int
     state: IssueState
-    state_reason: Optional[StateReason]
+    state_reason: StateReason | None
     title: str
-    user: Optional[SimpleUser]
+    user: SimpleUser | None
     labels: List[Label]
-    assignee: Optional[SimpleUser]
-    assignees: Optional[List[SimpleUser]]
+    assignee: SimpleUser | None
+    assignees: List[SimpleUser] | None
     locked: bool
-    active_lock_reason: Optional[str]
+    active_lock_reason: str | None
     comments: int
-    closed_at: Optional[datetime]
-    created_at: Optional[datetime]
-    updated_at: Optional[datetime]
+    closed_at: datetime | None
+    created_at: datetime | None
+    updated_at: datetime | None
     author_association: AuthorAssociation
-    reactions: Optional[Reactions] = None
-    pull_request: Optional[PullRequest] = None
-    body_html: Optional[str] = None
-    body_text: Optional[str] = None
-    timeline_url: Optional[str] = None
-    body: Optional[str] = None
+    reactions: Reactions | None = None
+    pull_request: PullRequest | None = None
+    body_html: str | None = None
+    body_text: str | None = None
+    timeline_url: str | None = None
+    body: str | None = None
 
 
 @dataclass

benchmarks/benchmarks/gh_issues/bench_asdict.py

@@ -1,6 +1,6 @@
 from dataclasses import asdict, dataclass
 from datetime import datetime
-from typing import List, Optional
+from typing import List
 
 from benchmarks.gh_issues.common import AuthorAssociation, IssueState, StateReason, create_response
 from benchmarks.pybench.bench_api import benchmark_plan
@@ -12,7 +12,7 @@ class SimpleUser:
     id: int
     node_id: str
     avatar_url: str
-    gravatar_id: Optional[str]
+    gravatar_id: str | None
     url: str
     html_url: str
     followers_url: str
@@ -26,9 +26,9 @@ class SimpleUser:
     received_events_url: str
     type: str
     site_admin: bool
-    name: Optional[str] = None
-    email: Optional[str] = None
-    starred_at: Optional[datetime] = None
+    name: str | None = None
+    email: str | None = None
+    starred_at: datetime | None = None
 
 
 @dataclass
@@ -37,7 +37,7 @@ class Label:
     node_id: str
     url: str
     name: str
-    description: Optional[str]
+    description: str | None
     color: str
     default: bool
 
@@ -58,11 +58,11 @@ class Reactions:
 
 @dataclass
 class PullRequest:
-    diff_url: Optional[str]
-    html_url: Optional[str]
-    patch_url: Optional[str]
-    url: Optional[str]
-    merged_at: Optional[datetime] = None
+    diff_url: str | None
+    html_url: str | None
+    patch_url: str | None
+    url: str | None
+    merged_at: datetime | None = None
 
 
 @dataclass
@@ -77,25 +77,25 @@ class Issue:
     html_url: str
     number: int
     state: IssueState
-    state_reason: Optional[StateReason]
+    state_reason: StateReason | None
     title: str
-    user: Optional[SimpleUser]
+    user: SimpleUser | None
     labels: List[Label]
-    assignee: Optional[SimpleUser]
-    assignees: Optional[List[SimpleUser]]
+    assignee: SimpleUser | None
+    assignees: List[SimpleUser] | None
     locked: bool
-    active_lock_reason: Optional[str]
+    active_lock_reason: str | None
     comments: int
-    closed_at: Optional[datetime]
-    created_at: Optional[datetime]
-    updated_at: Optional[datetime]
+    closed_at: datetime | None
+    created_at: datetime | None
+    updated_at: datetime | None
     author_association: AuthorAssociation
-    reactions: Optional[Reactions] = None
-    pull_request: Optional[PullRequest] = None
-    body_html: Optional[str] = None
-    body_text: Optional[str] = None
-    timeline_url: Optional[str] = None
-    body: Optional[str] = None
+    reactions: Reactions | None = None
+    pull_request: PullRequest | None = None
+    body_html: str | None = None
+    body_text: str | None = None
+    timeline_url: str | None = None
+    body: str | None = None
 
 
 @dataclass