bump clang-tidy action up to 19

also sync some of the checks with Castro

.clang-tidy

@@ -25,7 +25,6 @@ Checks: >
      modernize-*,
        -modernize-avoid-c-arrays,
        -modernize-use-trailing-return-type,
-       -modernize-use-using,
      performance-*,
        -performance-avoid-endl,
        -performance-enum-size,
@@ -34,17 +33,18 @@ Checks: >
        -readability-avoid-const-params-in-decls,
        -readability-braces-around-statements,
        -readability-else-after-return,
+       -readability-enum-initial-value,
        -readability-function-cognitive-complexity,
        -readability-function-size,
        -readability-identifier-length,
        -readability-implicit-bool-conversion,
        -readability-isolate-declaration,
        -readability-magic-numbers,
+       -readability-math-missing-parentheses,
        -readability-named-parameter,
        -readability-simplify-boolean-expr,
        -readability-static-accessed-through-instance,
      mpi-*,
      openmp-*,
-       -openmp-use-default-none
 
 HeaderFilterRegex: '(/Source/*|/Source/param/*|/Util/model_parser|/Util/simple_log|^\./|^tmp_build_dir/maestroex_sources/*/).*\.H$'

.github/workflows/clang-tidy.yml

@@ -25,7 +25,7 @@ jobs:
 
       - name: Install dependencies
         run: |
-          .github/workflows/dependencies_clang-tidy-apt-llvm.sh 17
+          .github/workflows/dependencies_clang-tidy-apt-llvm.sh 19
 
       - name: Compile wdconvect
         run: |