Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
D
DD4hep
Manage
Activity
Members
Code
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
cepc
externals
mirroring
DD4hep
Commits
636d2852
Commit
636d2852
authored
2 years ago
by
Andre Sailer
Browse files
Options
Downloads
Patches
Plain Diff
CI: add clang-tidy workflow for header_guards
parent
836bddf7
No related branches found
No related tags found
No related merge requests found
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
.github/workflows/clang-tidy.yml
+45
-0
45 additions, 0 deletions
.github/workflows/clang-tidy.yml
with
45 additions
and
0 deletions
.github/workflows/clang-tidy.yml
0 → 100644
+
45
−
0
View file @
636d2852
name
:
clang-tidy-guards
on
:
[
push
,
pull_request
]
jobs
:
default
:
runs-on
:
ubuntu-latest
strategy
:
fail-fast
:
false
matrix
:
LCG
:
[
"
LCG_102/x86_64-centos7-clang12-opt"
]
steps
:
-
uses
:
actions/checkout@v2
-
uses
:
cvmfs-contrib/github-action-cvmfs@v2
-
uses
:
aidasoft/run-lcg-view@v3
with
:
release-platform
:
${{ matrix.LCG }}
run
:
|
mkdir build
cd build
unset CPATH
echo "::group::CMakeConfig"
cmake -GNinja \
-DCMAKE_EXPORT_COMPILE_COMMANDS=ON \
-DDD4HEP_USE_GEANT4=ON \
-DBoost_NO_BOOST_CMAKE=ON \
-DDD4HEP_USE_LCIO=ON \
-DDD4HEP_USE_EDM4HEP=ON \
-DDD4HEP_USE_TBB=ON \
-DDD4HEP_USE_HEPMC3=ON \
-DDD4HEP_BUILD_DEBUG=OFF \
-DBUILD_TESTING=ON \
-DDD4HEP_DEBUG_CMAKE=ON \
-DDD4HEP_USE_XERCESC=ON \
-DCMAKE_BUILD_TYPE=Release \
-DCMAKE_CXX_FLAGS="-fdiagnostics-color=always" \
-DCMAKE_CXX_STANDARD=17 ..
echo "::group::Tidy"
cd ..
HEADER_FILES=$(find */include/* -name "*.h" | grep -v "/detail/\|/src/")
clang-tidy -p build \
--extra-arg "-DDD4HEP_MUST_USE_DETECTORIMP_H" \
--warnings-as-errors=".*" \
-checks=-*,llvm-header-guard \
${HEADER_FILES}
This diff is collapsed.
Click to expand it.
Preview
0%
Loading
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment