Skip to content

Commit 43781b0

Browse files
committed
ci linter fix
1 parent 158bf85 commit 43781b0

1 file changed

Lines changed: 21 additions & 38 deletions

File tree

.github/workflows/linter.yml

Lines changed: 21 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -1,61 +1,41 @@
1-
name: linter
21

2+
name: linter
3+
34
on:
45
push:
56
branches:
67
- main
78
pull_request:
8-
9+
910
jobs:
10-
linter-build:
11-
runs-on: ubuntu-latest
12-
13-
steps:
14-
- uses: actions/checkout@v4
15-
16-
- run: |
17-
sudo apt update
18-
19-
- run: |
20-
sudo apt update && sudo apt install -y \
21-
clang \
22-
llvm \
23-
lldb \
24-
gcc \
25-
g++ \
26-
cppcheck \
27-
clang-tools \
28-
libclang-dev \
29-
clang-format \
30-
python3-dev
31-
3211
linter-formatter:
33-
needs: linter-build
3412
runs-on: ubuntu-latest
3513
steps:
3614
- uses: actions/checkout@v4
37-
- run: |
38-
cd src && find . \( -name "*.c" -o -name "*.h" \) -exec clang-format -i {} \;
39-
15+
with:
16+
submodules: recursive
17+
- name: Install clang-format
18+
run: sudo apt update && sudo apt install -y clang-format
19+
- name: Run clang-format
20+
run: cd src && find . \( -name "*.c" -o -name "*.h" \) -exec clang-format -i {} \;
21+
4022
linter-aggressive_flags:
41-
needs: linter-build
4223
runs-on: ubuntu-latest
4324
steps:
4425
- uses: actions/checkout@v4
4526
with:
4627
submodules: recursive
47-
48-
- name: Install dependencies
28+
- name: Install LLVM 21
4929
run: |
5030
wget https://apt.llvm.org/llvm.sh
5131
chmod +x llvm.sh
5232
sudo ./llvm.sh 21
5333
sudo apt install -y clang-21 python3-dev
5434
- name: Build liburing
5535
run: cd requirements/liburing && make
56-
57-
- run: |
58-
CC=clang CFLAGS="-O0 -g3 \
36+
- name: Build with aggressive flags
37+
run: |
38+
CC=clang-21 CFLAGS="-O0 -g3 \
5939
-Wall \
6040
-Wextra \
6141
-Werror \
@@ -79,14 +59,17 @@ jobs:
7959
-isystem requirements/liburing/src \
8060
-isystem requirements/liburing" \
8161
python3 setup.py build_ext --inplace
82-
62+
8363
linter-cppcheck:
84-
needs: linter-build
8564
runs-on: ubuntu-latest
8665
steps:
8766
- uses: actions/checkout@v4
88-
- run: sudo apt update && sudo apt install -y cppcheck
89-
- run: |
67+
with:
68+
submodules: recursive
69+
- name: Install cppcheck
70+
run: sudo apt update && sudo apt install -y cppcheck
71+
- name: Run cppcheck
72+
run: |
9073
mkdir -p cpp-check-results
9174
cppcheck \
9275
--cppcheck-build-dir=cpp-check-results \

0 commit comments

Comments
 (0)