Skip to content

Commit a14f421

Browse files
LessUpqwencoder
andcommitted
fix(ci): simplify clang-format check and cmake config
- Remove check-path and exclude-regex from clang-format-action (action now checks entire repo by default, avoids path parsing issues) - Format cmake Configure command as multiline for readability (prevents shell semicolon parsing issues with CMAKE_CUDA_ARCHITECTURES) Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com> Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
1 parent ab3c79b commit a14f421

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

.github/workflows/ci.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,6 @@ jobs:
2929
uses: jidicula/clang-format-action@v4.14.0
3030
with:
3131
clang-format-version: '17'
32-
check-path: '.'
33-
exclude-regex: '(^|/)(build|third_party|external|vendor|node_modules|\.git|\.kiro)(/|$)'
3432
fallback-style: 'LLVM'
3533

3634
cuda-build:
@@ -49,7 +47,11 @@ jobs:
4947
rm -rf /var/lib/apt/lists/*
5048
5149
- name: Configure
52-
run: cmake -S . -B build -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTS=OFF -DCMAKE_CUDA_ARCHITECTURES="70;75;80;86"
50+
run: >
51+
cmake -S . -B build
52+
-DCMAKE_BUILD_TYPE=Release
53+
-DBUILD_TESTS=OFF
54+
-DCMAKE_CUDA_ARCHITECTURES="70;75;80;86"
5355
5456
- name: Build
5557
run: cmake --build build --target sgemm_benchmark -j$(nproc)

0 commit comments

Comments
 (0)