Merge pull request #2028 from gcmarx/remove-type-assertions

Byron · web-flow · commit bf5160951261 · 2025-05-23T08:49:00.000+02:00
remove type assertions from util.py
diff --git a/git/util.py b/git/util.py
@@ -1200,8 +1200,6 @@ def __getattr__(self, attr: str) -> T_IterableObj:
 return list.__getattribute__(self, attr)
 
 def __getitem__(self, index: Union[SupportsIndex, int, slice, str]) -> T_IterableObj: # type: ignore[override]
- assert isinstance(index, (int, str, slice)), "Index of IterableList should be an int or str"
-
 if isinstance(index, int):
 return list.__getitem__(self, index)
 elif isinstance(index, slice):
@@ -1214,8 +1212,6 @@ def __getitem__(self, index: Union[SupportsIndex, int, slice, str]) -> T_Iterabl
 # END handle getattr
 
 def __delitem__(self, index: Union[SupportsIndex, int, slice, str]) -> None:
- assert isinstance(index, (int, str)), "Index of IterableList should be an int or str"
-
 delindex = cast(int, index)
 if not isinstance(index, int):
 delindex = -1

-Original file line number
+Diff line change
 returnlist.__getattribute__(self, attr)
 def__getitem__(self, index: Union[SupportsIndex, int, slice, str]) ->T_IterableObj: # type: ignore[override]
 -assertisinstance(index, (int, str, slice)), "Index of IterableList should be an int or str"
+-
 ifisinstance(index, int):
 returnlist.__getitem__(self, index)
 elifisinstance(index, slice):
 # END handle getattr
 def__delitem__(self, index: Union[SupportsIndex, int, slice, str]) ->None:
 -assertisinstance(index, (int, str)), "Index of IterableList should be an int or str"
+-
 delindex=cast(int, index)
 ifnotisinstance(index, int):
 delindex=-1