Fix the first stage of mypy stricter errors

kbaikov · kbaikov · commit ddceff9b7709 · 2025-08-06T07:39:19.000+02:00
Following this doc: https://mypy.readthedocs.io/en/stable/existing_code.html#getting-to-strict Total 13 error fixed.
diff --git a/pygit2/blame.py b/pygit2/blame.py
@@ -127,7 +127,7 @@ def __del__(self) -> None:
         C.git_blame_free(self._blame)
 
     def __len__(self) -> int:
-        return C.git_blame_get_hunk_count(self._blame)  # type: ignore[no-any-return]
+        return C.git_blame_get_hunk_count(self._blame)
 
     def __getitem__(self, index: int) -> BlameHunk:
         chunk = C.git_blame_get_hunk_byindex(self._blame, index)
diff --git a/pygit2/callbacks.py b/pygit2/callbacks.py
@@ -414,7 +414,7 @@ def git_proxy_options(
     elif type(proxy) is str:
         opts.type = C.GIT_PROXY_SPECIFIED
         # Keep url in memory, otherwise memory is freed and bad things happen
-        payload.__proxy_url = ffi.new('char[]', to_bytes(proxy))  # type: ignore[attr-defined, no-untyped-call]
+        payload.__proxy_url = ffi.new('char[]', to_bytes(proxy))  # type: ignore[attr-defined]
         opts.url = payload.__proxy_url  # type: ignore[attr-defined]
     else:
         raise TypeError('Proxy must be None, True, or a string')
diff --git a/pygit2/ffi.py b/pygit2/ffi.py
@@ -24,5 +24,5 @@
 # Boston, MA 02110-1301, USA.
 
 # Import from pygit2
-from ._libgit2 import ffi  # type: ignore # noqa: F401
+from ._libgit2 import ffi  # noqa: F401
 from ._libgit2 import lib as C  # type: ignore # noqa: F401
diff --git a/pygit2/repository.py b/pygit2/repository.py
@@ -615,7 +615,7 @@ def diff(
 
         # Case 2: Index to workdir
         elif a is None and b is None:
-            return self.index.diff_to_workdir(**options)  # type: ignore[arg-type]
+            return self.index.diff_to_workdir(**options)
 
         # Case 3: Diff tree to index or workdir
         elif isinstance(a, Tree) and b is None:
diff --git a/test/test_index.py b/test/test_index.py
@@ -49,8 +49,8 @@ def test_read(testrepo: Repository) -> None:
     assert len(index) == 2
 
     with pytest.raises(TypeError):
-        index[()]  # type: ignore
-    utils.assertRaisesWithArg(ValueError, -4, lambda: index[-4])  # type: ignore
+        index[()]
+    utils.assertRaisesWithArg(ValueError, -4, lambda: index[-4])
     utils.assertRaisesWithArg(KeyError, 'abc', lambda: index['abc'])
 
     sha = 'a520c24d85fbfc815d385957eed41406ca5a860b'
diff --git a/test/test_tag.py b/test/test_tag.py
@@ -59,7 +59,7 @@ def test_new_tag(barerepo: Repository) -> None:
     target_prefix = target[:5]
     too_short_prefix = target[:3]
     with pytest.raises(ValueError):
-        barerepo.create_tag(name, too_short_prefix, ObjectType.BLOB, tagger, message)  # type: ignore
+        barerepo.create_tag(name, too_short_prefix, ObjectType.BLOB, tagger, message)
 
     sha = barerepo.create_tag(name, target_prefix, ObjectType.BLOB, tagger, message)
     tag = barerepo[sha]
diff --git a/test/test_tree.py b/test/test_tree.py
@@ -99,7 +99,7 @@ def test_sorting(barerepo: Repository) -> None:
     tree_a = barerepo['18e2d2e9db075f9eb43bcb2daa65a2867d29a15e']
     assert isinstance(tree_a, Tree)
     assert list(tree_a) == sorted(reversed(list(tree_a)), key=pygit2.tree_entry_key)
-    assert list(tree_a) != reversed(list(tree_a))  # type: ignore[comparison-overlap]
+    assert list(tree_a) != reversed(list(tree_a))
 
 
 def test_read_subtree(barerepo: Repository) -> None:
