Skip to content

Commit

Permalink
Fix clang tidy ci (#886)
Browse files Browse the repository at this point in the history
  • Loading branch information
mcbarton authored May 8, 2024
1 parent aec1e02 commit 3f0e5a1
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 5 deletions.
12 changes: 8 additions & 4 deletions .github/workflows/clang-tidy-review.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,18 +22,22 @@ jobs:
with:
version: "16.0.0"

- name: install lit
run: pip install lit

- name: run git config command
run: |
git config --global --add safe.directory /github/workspace
- name: Run clang-tidy
uses: ZedThree/[email protected]
id: review
with:
build_dir: build
apt_packages: libxml2,libxml2-dev,libtinfo-dev,zlib1g-dev,libzstd-dev
apt_packages: cmake,libxml2,libxml2-dev,libtinfo-dev,zlib1g-dev,libzstd-dev
exclude: "test/*,unittests/*,benchmark/*,demos/*"
split_workflow: true
cmake_command: >
pip install cmake lit &&
cmake --version &&
git config --global --add safe.directory /github/workspace &&
cmake . -B build -DLLVM_DIR="$GITHUB_WORKSPACE/llvm"
-DClang_DIR="$GITHUB_WORKSPACE/llvm"
-DCMAKE_BUILD_TYPE="Release"
Expand Down
1 change: 0 additions & 1 deletion include/clad/Differentiator/Compatibility.h
Original file line number Diff line number Diff line change
Expand Up @@ -389,7 +389,6 @@ getConstantArrayType(const ASTContext& Ctx, QualType EltTy,
#define CLAD_COMPAT_CLANG11_UnaryOperator_ExtraParams ,Node->getFPOptionsOverride()
#endif


// Clang 12 rename DeclaratorContext::LambdaExprContext to DeclaratorContext::LambdaExpr.
// Clang 15 add one extra param to clang::Declarator() - const ParsedAttributesView & DeclarationAttrs

Expand Down

0 comments on commit 3f0e5a1

Please sign in to comment.