CI: add clang-tidy job - #659
Conversation
- clang-tidy is not implemented in the alpaka CMake yet
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
| ) | ||
|
|
||
| if [[ "${APCI_CLANG_TIDY}" == "ON" ]]; then | ||
| clang_apt_package_list+=("clang-tidy-${compiler_version}") |
There was a problem hiding this comment.
I'm not sure, if this apt package is enough.
It installs the executable clang-tidy-<version> (e.g. clang-tidy-21). Maybe we need to create the file link clang-tidy, which points to clang-tidy-<version>, that CMake can find it.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #659 +/- ##
=======================================
Coverage 73.09% 73.09%
=======================================
Files 123 123
Lines 3334 3334
Branches 917 917
=======================================
Hits 2437 2437
Misses 145 145
Partials 752 752
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
No description provided.