Modernize type annotations

dandi/cli/cmd_move.py

@@ -1,7 +1,5 @@
 from __future__ import annotations
 
-from typing import Optional
-
 import click
 
 from .base import devel_debug_option, instance_option, map_to_click_exceptions
@@ -47,10 +45,10 @@
 @map_to_click_exceptions
 def move(
     paths: tuple[str],
-    dandiset: Optional[str],
+    dandiset: str | None,
     dry_run: bool,
     existing: str,
-    jobs: Optional[int],
+    jobs: int | None,
     regex: bool,
     work_on: str,
     dandi_instance: str,

dandi/cli/cmd_validate.py

@@ -4,7 +4,7 @@
 import logging
 import os
 import re
-from typing import List, Optional, cast
+from typing import cast
 import warnings
 
 import click
@@ -90,10 +90,10 @@ def validate_bids(
 @map_to_click_exceptions
 def validate(
     paths: tuple[str, ...],
-    ignore: Optional[str],
+    ignore: str | None,
     grouping: str,
     min_severity: str,
-    schema: Optional[str] = None,
+    schema: str | None = None,
     devel_debug: bool = False,
     allow_any_path: bool = False,
 ) -> None:
@@ -139,7 +139,7 @@ def validate(
 def _process_issues(
     validator_result: Iterable[ValidationResult],
     grouping: str,
-    ignore: Optional[str] = None,
+    ignore: str | None = None,
 ) -> None:
     issues = [i for i in validator_result if i.severity is not None]
     if ignore is not None:
@@ -149,7 +149,7 @@ def _process_issues(
         display_errors(
             purviews,
             [i.id for i in issues],
-            cast(List[Severity], [i.severity for i in issues]),
+            cast("list[Severity]", [i.severity for i in issues]),
             [i.message for i in issues],
         )
     elif grouping == "path":
@@ -161,7 +161,7 @@ def _process_issues(
             display_errors(
                 [purview],
                 [i.id for i in applies_to],
-                cast(List[Severity], [i.severity for i in applies_to]),
+                cast("list[Severity]", [i.severity for i in applies_to]),
                 [i.message for i in applies_to],
             )
     else:
@@ -185,10 +185,10 @@ def _get_severity_color(severities: list[Severity]) -> str:
 
 
 def display_errors(
-    purviews: list[Optional[str]],
+    purviews: list[str | None],
     errors: list[str],
     severities: list[Severity],
-    messages: list[Optional[str]],
+    messages: list[str | None],
 ) -> None:
     """
     Unified error display for validation CLI, which auto-resolves grouping

dandi/conftest.py

@@ -1,4 +1,4 @@
-from typing import List
+from __future__ import annotations
 
 from _pytest.config import Config
 from _pytest.config.argparsing import Parser
@@ -16,7 +16,7 @@ def pytest_addoption(parser: Parser) -> None:
     )
 
 
-def pytest_collection_modifyitems(items: List[Item], config: Config) -> None:
+def pytest_collection_modifyitems(items: list[Item], config: Config) -> None:
     # Based on <https://pythontesting.net/framework/pytest/pytest-run-tests
     # -using-particular-fixture/>
     if config.getoption("--dandi-api"):

dandi/consts.py

@@ -4,7 +4,6 @@
 from dataclasses import dataclass
 from enum import Enum
 import os
-from typing import Optional
 
 #: A list of metadata fields which dandi extracts from .nwb files.
 #: Additional fields (such as ``number_of_*``) might be added by
@@ -99,7 +98,7 @@ class EmbargoStatus(Enum):
 @dataclass(frozen=True)
 class DandiInstance:
     name: str
-    gui: Optional[str]
+    gui: str | None
     api: str
 
     @property