diff mercurial/interfaces/repository.py @ 52487:3daaa5195a30

typing: align the signatures of `repository.ifilestorage` overrides This is the same as 048c11993d6a for `imanifestrevisionstored`, with the same belated realization about missing the Protocol marker. Since I've been jumping around to the various subclasses and copying extant hints to places where they're missing, do that here too. This also covers the 3 interfaces subclassed by `repository.ifilestorage` (`repository.ifileindex`, `repository.ifiledata`, and `repository.ifilemutation`). Obviously the whole base class needs hints, but I don't want to get side tracked here- it doesn't look like any of the implementing classes have any additional hints.
author Matt Harbison <matt_harbison@yahoo.com>
date Wed, 23 Oct 2024 12:21:52 -0400
parents c0d9fda9f5f5
children 8c89e978375c
line wrap: on
line diff
--- a/mercurial/interfaces/repository.py	Tue Dec 10 23:36:14 2024 -0500
+++ b/mercurial/interfaces/repository.py	Wed Oct 23 12:21:52 2024 -0400
@@ -15,6 +15,8 @@
     Any,
     Callable,
     Collection,
+    Iterable,
+    Iterator,
     Mapping,
     Protocol,
 )
@@ -608,10 +610,10 @@
     nullid: bytes
     """node for the null revision for use as delta base."""
 
-    def __len__(self):
+    def __len__(self) -> int:
         """Obtain the number of revisions stored for this file."""
 
-    def __iter__(self):
+    def __iter__(self) -> Iterator[int]:
         """Iterate over revision numbers for this file."""
 
     def hasnode(self, node):
@@ -967,7 +969,7 @@
         callers are expected to handle this special value.
         """
 
-    def verifyintegrity(self, state):
+    def verifyintegrity(self, state) -> Iterable[iverifyproblem]:
         """Verifies the integrity of file storage.
 
         ``state`` is a dict holding state of the verifier process. It can be