enhance: Update mergify check more generally

- fix: Remove not useful SequenceIterator

See also: #2612

Signed-off-by: yangxuan <[email protected]>

Author: XuanYang-cn

.github/mergify.yml

@@ -4,14 +4,14 @@ pull_request_rules:
       - or:
         - base=master
         - base~=2\.\d
-      - "status-success=Run Python Tests (3.8, windows-2022)"
-      - "status-success=Run Python Tests (3.13, windows-2022)"
-      - "status-success=Run Python Tests (3.8, ubuntu-22.04)"
-      - "status-success=Run Python Tests (3.13, ubuntu-22.04)"
-      - "status-success=Run Check Proto (3.8)"
-      - "status-success=Code lint check (3.8)"
-      - "status-success=Run Check Proto (3.13)"
-      - "status-success=Code lint check (3.13)"
+      # Require that Python tests succeed on Windows (any Python version, any Windows build)
+      - "status-success~=Run Python Tests \\(\\d+\\.\\d+, windows-\\S+\\)"
+      # Require that Python tests succeed on Ubuntu (any Python version, any Ubuntu build)
+      - "status-success~=Run Python Tests \\(\\d+\\.\\d+, ubuntu-\\S+\\)"
+      # Require that proto checks pass for any Python version
+      - "status-success~=Run Check Proto \\(\\d+\\.\\d+\\)"
+      # Require that code lint checks pass for any Python version
+      - "status-success~=Code lint check \\(\\d+\\.\\d+\\)"
     actions:
       label:
         add:

.github/workflows/pull_request.yml

@@ -10,7 +10,7 @@ jobs:
     name: Run Python Tests
     strategy:
       matrix:
-        python-version: [3.8, 3.12]
+        python-version: [3.8, 3.13]
         os: [ubuntu-22.04, windows-2022]
     runs-on: ${{ matrix.os }}
 

pymilvus/client/abstract.py

@@ -1,5 +1,5 @@
 import abc
-from typing import Any, Dict, List, Optional, Sequence, Tuple, Union
+from typing import Any, Dict, List, Optional, Tuple, Union
 
 import ujson
 
@@ -351,19 +351,6 @@ def _pack(self, raw: Any):
         )
 
 
-class SequenceIterator:
-    def __init__(self, seq: Sequence[Any]):
-        self._seq = seq
-        self._idx = 0
-
-    def __next__(self) -> Any:
-        if self._idx < len(self._seq):
-            res = self._seq[self._idx]
-            self._idx += 1
-            return res
-        raise StopIteration
-
-
 class BaseRanker:
     def __int__(self):
         return
@@ -660,9 +647,6 @@ def get_fields_by_range(
                 continue
         return field2data
 
-    def __iter__(self) -> SequenceIterator:
-        return SequenceIterator(self)
-
     def __str__(self) -> str:
         """Only print at most 10 query results"""
         reminder = f" ... and {len(self) - 10} results remaining" if len(self) > 10 else ""
@@ -743,9 +727,6 @@ def __init__(
 
         super().__init__(hits)
 
-    def __iter__(self) -> SequenceIterator:
-        return SequenceIterator(self)
-
     def __str__(self) -> str:
         """Only print at most 10 query results"""
         reminder = f" ... and {len(self) - 10} entities remaining" if len(self) > 10 else ""