Skip to content

Commit 213c9a6

Browse files
authored
Fix clang tidy (#67)
1 parent 5afa4a7 commit 213c9a6

File tree

2 files changed

+21
-12
lines changed

2 files changed

+21
-12
lines changed

.github/workflows/clang-tidy-review.yml

Lines changed: 21 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,20 @@ concurrency:
1414
jobs:
1515
review:
1616
runs-on: ubuntu-latest
17+
1718
steps:
1819
- name: Checkout PR branch
1920
uses: actions/checkout@v4
2021

22+
- name: Setup Python
23+
uses: actions/setup-python@v5
24+
with:
25+
python-version: "3.11"
26+
2127
- name: Install LLVM and Clang
22-
uses: KyleMayes/install-llvm-action@v2
28+
uses: KyleMayes/install-llvm-action@v2.0.2
2329
with:
24-
version: "18.1.3"
30+
version: "16.0.0"
2531

2632
- name: Run clang-tidy
2733
uses: ZedThree/[email protected]
@@ -32,15 +38,19 @@ jobs:
3238
exclude: "test/*,unittests/*,benchmark/*,demos/*"
3339
split_workflow: true
3440
cmake_command: >
35-
set -x &&
36-
mkdir micromamba && cd micromamba && curl -Ls https://micro.mamba.pm/api/micromamba/linux-64/latest | tar -xvj bin/micromamba &&
37-
export MAMBA_ROOT_PREFIX=/github/workspace/micromamba &&
38-
eval "$(./bin/micromamba shell hook -s posix)" &&
39-
micromamba create -n xeus-cpp -y --log-level warning -f /github/workspace/environment-dev.yml &&
40-
micromamba activate xeus-cpp &&
41-
cd .. &&
42-
git config --global --add safe.directory /github/workspace &&
41+
cmake . -B build -DCMAKE_EXPORT_COMPILE_COMMANDS=On || true &&
42+
rm -rf build &&
43+
set -x &&
44+
mkdir micromamba &&
45+
cd micromamba &&
46+
curl -Ls https://micro.mamba.pm/api/micromamba/linux-64/latest | tar -xvj bin/micromamba &&
47+
export MAMBA_ROOT_PREFIX=$GITHUB_WORKSPACE/micromamba &&
48+
eval "$(./bin/micromamba shell hook -s posix)" &&
49+
micromamba create -n xeus-cpp -y --log-level warning -f $GITHUB_WORKSPACE/environment-dev.yml &&
50+
micromamba activate xeus-cpp &&
51+
cd .. &&
52+
git config --global --add safe.directory /github/workspace &&
4353
cmake . -B build -DCMAKE_EXPORT_COMPILE_COMMANDS=On
44-
54+
4555
- name: Upload artifacts
4656
uses: ZedThree/clang-tidy-review/[email protected]

include/xeus-cpp/xinterpreter_wasm.hpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
* The full license is in the file LICENSE, distributed with this software. *
88
************************************************************************************/
99

10-
1110
#ifndef XEUS_CPP_INTERPRETER_WASM_HPP
1211
#define XEUS_CPP_INTERPRETER_WASM_HPP
1312

0 commit comments

Comments
 (0)