Sorted includes and parametric types, added StaticArrays package incl…

…ude in CI
AIDASoft · Sep 1, 2023 · 2421a42 · 2421a42
1 parent 84c01e9
commit 2421a42
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 2 deletions.
diff --git a/.github/workflows/sanitizers.yaml b/.github/workflows/sanitizers.yaml
@@ -51,3 +51,6 @@ jobs:
             echo "::group::Run tests"
             ctest --output-on-failure
             echo "::endgroup::"
+
+      - name: Install StaticArrays Julia Package
+        run: julia -e 'import Pkg; Pkg.add("StaticArrays")'
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -53,3 +53,6 @@ jobs:
           echo "::group::Install"
           ninja install
           echo "::endgroup::"
+
+    - name: Install StaticArrays Julia Package
+      run: julia -e 'import Pkg; Pkg.add("StaticArrays")'
diff --git a/.github/workflows/ubuntu.yml b/.github/workflows/ubuntu.yml
@@ -44,3 +44,6 @@ jobs:
           echo "::group::Install"
           ninja install
           echo "::endgroup::"
+
+    - name: Install StaticArrays Julia Package
+      run: julia -e 'import Pkg; Pkg.add("StaticArrays")'
diff --git a/python/podio/generator_utils.py b/python/podio/generator_utils.py
@@ -187,7 +187,7 @@ def __init__(self, name, **kwargs):
 
     self.julia_type = get_julia_type(cpp_type=self.bare_type, is_array=self.is_array,
                                      array_type=self.array_type, array_size=self.array_size)
-  
+
   @property
   def docstring(self):
     """Docstring to be used in code generation"""

diff --git a/python/podio_class_generator.py b/python/podio_class_generator.py
@@ -291,7 +291,7 @@ def _preprocess_for_julia(self, datatype):
       if not member.is_builtin:
         includes_jl_struct.add(self._build_julia_include(member))
     datatype['includes_jl']['struct'].update((includes_jl_struct))
-    sorted(datatype['includes_jl']['struct'])
+    datatype['includes_jl']['struct'] = sorted(datatype['includes_jl']['struct'])
 
   @staticmethod
   def _get_julia_params(datatype):