ci: resolve intermittent setup failure caused by venv downgrade race (#2620)

Author: haakonvt

.github/actions/setup/action.yml

@@ -22,10 +22,7 @@ runs:
       uses: actions/cache@5a3ec84eff668545956fd18022155c47e93e2684 # v4
       with:
         path: |
-          ~/.cache/pypoetry
-          ~/.poetry
-          !~/.cache/pypoetry/artifacts
-          !~/.cache/pypoetry/repositories
+          ~/.cache/pypoetry/cache
         key: ${{ runner.os }}-poetry-${{ hashFiles('**/poetry.lock') }}-${{ inputs.python_version }}-${{ inputs.extras }}
 
     - name: Install dependencies

.github/workflows/conventional-pr-title.yml

@@ -12,13 +12,3 @@ jobs:
       - uses: amannn/action-semantic-pull-request@e32d7e603df1aa1ba07e981f2a23455dee596825 # v5
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        with:
-          types: |
-            feat
-            fix
-            chore
-            docs
-            style
-            refactor
-            perf
-            test

cognite/client/_api/datapoint_tasks.py

@@ -170,7 +170,7 @@ def _parse(
             # We merge 'defaults' and the given user query; the query takes precedence:
             if isinstance(query, exp_type):
                 id_dct = {arg_name: query}
-                query = DatapointsQuery(**self.top_level_defaults, **id_dct)  # type: ignore [misc, arg-type]
+                query = DatapointsQuery(**self.top_level_defaults, **id_dct)  # type: ignore [arg-type]
 
             elif isinstance(query, DatapointsQuery):
                 if query.identifier.name() != arg_name:

cognite/client/config.py

@@ -4,7 +4,7 @@
 import pprint
 import re
 import warnings
-from typing import Any, NoReturn, overload
+from typing import Any, ClassVar, NoReturn, overload
 
 from cognite.client._version import __api_subversion__
 from cognite.client.credentials import CredentialProvider
@@ -48,6 +48,8 @@ class GlobalConfig:
             features. Defaults to False.
     """
 
+    _instance: ClassVar[GlobalConfig]
+
     def __new__(cls) -> GlobalConfig:
         if hasattr(cls, "_instance"):
             raise TypeError(

cognite/client/data_classes/capabilities.py

@@ -170,7 +170,7 @@ def from_tuple(cls, tpl: CapabilityTuple) -> Self:
         elif tpl.scope_extra is None:
             scope = scope_cls([tpl.scope_id])  # type: ignore [call-arg]
         elif scope_cls is TableScope:
-            scope = scope_cls({tpl.database: [tpl.table] if tpl.table else []})  # type: ignore [call-arg]
+            scope = scope_cls({tpl.database: [tpl.table] if tpl.table else []})
         else:
             raise ValueError(f"CapabilityTuple not understood: {tpl}")
 

cognite/client/data_classes/data_modeling/data_types.py

@@ -35,7 +35,7 @@ def load(cls, data: dict[str, str] | tuple[str, str] | DirectRelationReference |
                 return cls(space, xid)
             case (space, xid):
                 return cls(space, xid)
-            case cls() | NodeId():  # type: ignore [misc]
+            case cls() | NodeId():
                 return cls(data.space, data.external_id)
             case _:
                 raise ValueError("Invalid data provided to load method. Must be dict or tuple with two elements.")

cognite/client/utils/_json_extended.py

@@ -62,7 +62,7 @@ def dumps(
 
 def convert_to_float(value: float | str | None) -> float | None:
     if value.__class__ is str:  # like this abomination; faster than float(value)
-        return _FLOAT_API_MAPPING[value]  # type: ignore [index]
+        return _FLOAT_API_MAPPING[value]
     return value  # type: ignore [return-value]
 
 

poetry.lock

@@ -99,7 +99,7 @@ ipykernel = "^7.2.0"
 [tool.poetry.group.docs.dependencies]
 sphinx = ">=8.1"
 sphinx-rtd-theme = ">=1"
-sphinx-copybutton = "^0.5.2"    
+sphinx-copybutton = "^0.5.2"
 sphinx-autosummary-accessors = "^2025.3.1"
 
 [build-system]

pyproject.toml

@@ -2377,9 +2377,9 @@ def test_multiple_settings_for_ignore_unknown_ids(
             assert len(res_lst) == 3
             assert res_lst.get(external_id=random_xids[0]) is None
             dps_xid = res_lst.get(external_id=ts_str.external_id)
-            assert isinstance(dps_xid, exp_res_lst_type._RESOURCE)  # type: ignore[attr-defined]
+            assert isinstance(dps_xid, exp_res_lst_type._RESOURCE)
             dps_id = res_lst.get(id=ts_num.id)
-            assert isinstance(dps_id, exp_res_lst_type._RESOURCE)  # type: ignore[attr-defined]
+            assert isinstance(dps_id, exp_res_lst_type._RESOURCE)
 
     def test_query_with_duplicates(
         self,
@@ -2404,17 +2404,17 @@ def test_query_with_duplicates(
             )
             assert isinstance(res_lst, exp_res_lst_type)
             # Check non-duplicated in result:
-            assert isinstance(res_lst.get(id=ms_bursty_ts.id), exp_res_lst_type._RESOURCE)  # type: ignore[attr-defined]
-            assert isinstance(res_lst.get(external_id=ms_bursty_ts.external_id), exp_res_lst_type._RESOURCE)  # type: ignore[attr-defined]
+            assert isinstance(res_lst.get(id=ms_bursty_ts.id), exp_res_lst_type._RESOURCE)
+            assert isinstance(res_lst.get(external_id=ms_bursty_ts.external_id), exp_res_lst_type._RESOURCE)
             # Check duplicated in result:
             assert isinstance(res_lst.get(id=ts_numeric.id), list)
             assert isinstance(res_lst.get(id=ts_string.id), list)
             assert isinstance(res_lst.get(external_id=ts_numeric.external_id), list)
             assert isinstance(res_lst.get(external_id=ts_string.external_id), list)
-            assert len(res_lst.get(id=ts_numeric.id)) == 3  # type: ignore[arg-type]
-            assert len(res_lst.get(id=ts_string.id)) == 2  # type: ignore[arg-type]
-            assert len(res_lst.get(external_id=ts_numeric.external_id)) == 3  # type: ignore[arg-type]
-            assert len(res_lst.get(external_id=ts_string.external_id)) == 2  # type: ignore[arg-type]
+            assert len(res_lst.get(id=ts_numeric.id)) == 3
+            assert len(res_lst.get(id=ts_string.id)) == 2
+            assert len(res_lst.get(external_id=ts_numeric.external_id)) == 3
+            assert len(res_lst.get(external_id=ts_string.external_id)) == 2
 
 
 class TestRetrieveDataFrameAPI: