Skip to content

CI: Fix clang-tidy cache#4944

Merged
asalmgren merged 8 commits intoAMReX-Codes:developmentfrom
WeiqunZhang:clang-tidy-ccache
Feb 21, 2026
Merged

CI: Fix clang-tidy cache#4944
asalmgren merged 8 commits intoAMReX-Codes:developmentfrom
WeiqunZhang:clang-tidy-ccache

Conversation

@WeiqunZhang
Copy link
Copy Markdown
Member

ccache 4.12 has a slight new different format in its log. So We have to update our python script processing and generating makefile for clang-tidy.

Fix new warnings from clang-tidy-21. We missed them because we switched to ccache-4.12 before switching to clang-tidy-21.

Also enable clang-tidy for the C++20 test.

@WeiqunZhang
Copy link
Copy Markdown
Member Author

/run-hpsf-gitlab-ci

@github-actions
Copy link
Copy Markdown

@amrex-gitlab-ci-reporter
Copy link
Copy Markdown

GitLab CI 1422636 finished with status: failed. See details at https://gitlab.spack.io/amrex/amrex/-/pipelines/1422636.

WeiqunZhang and others added 7 commits February 11, 2026 19:57
ccache 4.12 has a slight new different format in its log. So We have to
update our python script processing and generating makefile for clang-tidy.

Fix new warnings from clang-tidy-21. We missed them because we switched to
ccache-4.12 before switching to clang-tidy-21.

Also enable clang-tidy for the C++20 test.
WeiqunZhang added a commit to WeiqunZhang/amrex that referenced this pull request Feb 12, 2026
ccache 4.12 has a slight new different format in its log. So We have to
update our python script processing and generating makefile for clang-tidy.

Fix new warnings from clang-tidy-21. We missed them because we switched to
ccache-4.12 before switching to clang-tidy-21.

Also enable clang-tidy for the C++20 test.
@asalmgren asalmgren merged commit 98bce06 into AMReX-Codes:development Feb 21, 2026
74 checks passed
@WeiqunZhang WeiqunZhang deleted the clang-tidy-ccache branch February 21, 2026 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants