celerity / celerity-runtime

High-level C++ for Accelerator Clusters
https://celerity.github.io
MIT License
139 stars 18 forks source link

CI: Let clang-tidy auto-detect config files #233

Closed psalz closed 9 months ago

psalz commented 9 months ago

clang-tidy-review v0.14.0 added support for this; it allows us to use different files for different folders.

Based on #232, so that should be merged first.

github-actions[bot] commented 9 months ago

Check-perf-impact results: (4c65f1399a47e0eb1340f63004745b17)

:question: No new benchmark data submitted. :question:
Please re-run the microbenchmarks and include the results if your commit could potentially affect performance.

psalz commented 9 months ago

Closing as I had to revert back to clang-tidy-review@0.9.0 for #232 (0.14.0 fails when installing pip dependencies on Ubuntu 22.04). Revisit this at some point.