Skip to content

Commit 3eed197

Browse files
committed
Fix clang tidy
1 parent 5afa4a7 commit 3eed197

File tree

2 files changed

+22
-14
lines changed

2 files changed

+22
-14
lines changed

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

Lines changed: 22 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -14,33 +14,42 @@ 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+
27+
- name: install mamba
28+
uses: mamba-org/setup-micromamba@main
29+
with:
30+
environment-file: environment-dev.yml
31+
init-shell: >-
32+
bash
33+
environment-name: xeus-cpp
34+
2135
- name: Install LLVM and Clang
22-
uses: KyleMayes/install-llvm-action@v2
36+
uses: KyleMayes/install-llvm-action@v2.0.2
2337
with:
24-
version: "18.1.3"
38+
version: "16.0.0"
39+
40+
- name: run git config command
41+
run: |
42+
git config --global --add safe.directory /github/workspace
2543
2644
- name: Run clang-tidy
2745
uses: ZedThree/[email protected]
28-
id: review
46+
2947
with:
3048
build_dir: build
3149
apt_packages: libxml2,libxml2-dev,libtinfo-dev,zlib1g-dev,libzstd-dev,curl
3250
exclude: "test/*,unittests/*,benchmark/*,demos/*"
3351
split_workflow: true
34-
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 &&
43-
cmake . -B build -DCMAKE_EXPORT_COMPILE_COMMANDS=On
52+
cmake_command: cmake . -B build -DCMAKE_PREFIX_PATH=/home/runner/micromamba/envs/xeus-cpp -DCMAKE_EXPORT_COMPILE_COMMANDS=On
4453

4554
- name: Upload artifacts
4655
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)