NCAR / micm

A model-independent chemistry module for atmosphere models
https://ncar.github.io/micm/
Apache License 2.0
6 stars 4 forks source link

518 integrate clang tidy workflow with config file #519

Closed mwaxmonsky closed 2 months ago

mwaxmonsky commented 2 months ago

Adding bugprone-* checks to clang-tidy config and using config instead of hardcoded list of checks in CI.

Needed to add custom .dockerignore for clang-tidy build due to .clang-tidy being ignored in main ingnore file.

clang-tidy runs fail with errors in workflow but this is due to using warnings as errors. The build of the source code produced no errors.

Closes #518

K20shores commented 2 months ago

Please add closing language if this resolves #518

mwaxmonsky commented 2 months ago

Please add closing language if this resolves #518

Good catch, thanks! Updated.