Skip to content

CI: add clang-tidy job - #659

Draft
SimeonEhrig wants to merge 1 commit into
alpaka-group:devfrom
SimeonEhrig:ciClangTidy
Draft

CI: add clang-tidy job#659
SimeonEhrig wants to merge 1 commit into
alpaka-group:devfrom
SimeonEhrig:ciClangTidy

Conversation

@SimeonEhrig

@SimeonEhrig SimeonEhrig commented Jul 28, 2026

Copy link
Copy Markdown
Member

No description provided.

- clang-tidy is not implemented in the alpaka CMake yet
@SimeonEhrig SimeonEhrig added this to the release-3.0 milestone Jul 28, 2026
@SimeonEhrig SimeonEhrig added refactoring This issue or pull request already exists feature New feature or request CI changing CI jobs labels Jul 28, 2026
@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: ab737ca4-9d5f-4be9-a4b4-e25d1c410b85

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

)

if [[ "${APCI_CLANG_TIDY}" == "ON" ]]; then
clang_apt_package_list+=("clang-tidy-${compiler_version}")

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

codecov Bot commented Jul 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.09%. Comparing base (ce2641f) to head (1d762d7).

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           
Flag Coverage Δ
cpu 73.09% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI changing CI jobs feature New feature or request refactoring This issue or pull request already exists

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant