Merge pull request #313 from PowerGridModel/feature/update-examples

Fix ci: github actions requires ci-passed + bump pre-commit + fix mypy

Author: mgovers

.github/workflows/ci.yml

@@ -52,8 +52,12 @@ jobs:
   ci-passed:
     runs-on: ubuntu-latest
     needs: [ci-started, build-test-release, check-code-quality, reuse-compliance]
+    if: always()
 
     steps:
+      # this explicit check is needed cfr. https://github.com/orgs/community/discussions/75568
+      - name: "Check whether all jobs passed"
+        run: echo '${{ toJSON(needs) }}' | jq -e 'to_entries | all(.value.result == "success")'
       - run: echo "ci passed"
 
   publish:

.pre-commit-config.yaml

@@ -4,19 +4,19 @@
 
 repos:
   - repo: https://github.com/fsfe/reuse-tool
-    rev: v4.0.3
+    rev: v5.0.2
     hooks:
       - id: reuse
   - repo: https://github.com/pycqa/isort
-    rev: 5.13.2
+    rev: 6.0.1
     hooks:
       - id: isort
   - repo: https://github.com/psf/black
     rev: 25.1.0
     hooks:
       - id: black-jupyter
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.11.2
+    rev: v1.16.0
     hooks:
       - id: mypy
         additional_dependencies: [numpy, pandas]

src/power_grid_model_io/converters/tabular_converter.py

@@ -265,6 +265,9 @@ def _convert_col_def_to_attribute(  # pylint: disable = too-many-arguments,too-m
         """
         # To avoid mistakes, the attributes in the mapping should exist. There is one extra attribute called
         # 'extra' in which extra information can be captured.
+        if pgm_data.dtype.names is None:
+            raise ValueError(f"pgm_data for '{component}s' has no attributes defined. (dtype.names is None)")
+
         if attr not in pgm_data.dtype.names and attr not in ["extra", "filters"]:
             attrs = ", ".join(pgm_data.dtype.names)
             raise KeyError(f"Could not find attribute '{attr}' for '{component}s'. (choose from: {attrs})")