repository: Handle linting issues

Signed-off-by: Jussi Kukkonen <jkukkonen@google.com>

Author: jku

examples/repository/_simplerepo.py

@@ -61,7 +61,9 @@ def __init__(self) -> None:
         self.target_cache: Dict[str, bytes] = {}
         # version cache for snapshot and all targets, updated in close()
         self._snapshot_info = MetaFile(1)
-        self._targets_infos = defaultdict(lambda: MetaFile(1))
+        self._targets_infos: Dict[str, MetaFile] = defaultdict(
+            lambda: MetaFile(1)
+        )
 
         # setup a basic repository, generate signing key per top-level role
         with self.edit("root", init=True) as root:

tuf/repository/_repository.py

@@ -3,10 +3,10 @@
 
 """Repository Abstraction for metadata management"""
 
-from copy import deepcopy
 import logging
 from abc import ABC, abstractmethod
 from contextlib import contextmanager, suppress
+from copy import deepcopy
 from typing import Dict, Generator, Optional, Tuple
 
 from tuf.api.metadata import Metadata, MetaFile, Signed
@@ -132,7 +132,8 @@ def snapshot(self, force: bool = False) -> Tuple[bool, Dict[str, MetaFile]]:
                 raise AbortEdit("Skip snapshot: No targets version changes")
 
         if not update_version:
-            logger.debug("Snapshot update not needed")
+            # this is reachable as edit() handles AbortEdit
+            logger.debug("Snapshot update not needed")  # type: ignore[unreachable]
         else:
             logger.debug(
                 "Snapshot v%d, %d targets", snapshot.version, len(snapshot.meta)
@@ -153,7 +154,7 @@ def timestamp(self, force: bool = False) -> Tuple[bool, Optional[MetaFile]]:
         removed = None
         with self.edit("timestamp") as timestamp:
             if self.snapshot_info.version < timestamp.snapshot_meta.version:
-                raise ValueError(f"snapshot version rollback")
+                raise ValueError("snapshot version rollback")
 
             if self.snapshot_info.version > timestamp.snapshot_meta.version:
                 update_version = True
@@ -164,7 +165,8 @@ def timestamp(self, force: bool = False) -> Tuple[bool, Optional[MetaFile]]:
                 raise AbortEdit("Skip timestamp: No snapshot version changes")
 
         if not update_version:
-            logger.debug("Timestamp update not needed")
+            # this is reachable as edit() handles AbortEdit
+            logger.debug("Timestamp update not needed")  # type: ignore[unreachable]
         else:
             logger.debug("Timestamp v%d", timestamp.version)
         return update_version, removed