Skip to content

Commit 3677ab5

Browse files
junjieqifacebook-github-bot
authored andcommitted
Switch clang-format-11 to clang-format-18 (#3372)
Summary: In this commit ab2b7f5, they changed format based on clang-format-18. However, we still use clang-format-11 in our circle ci job which caused the failure. In this PR, we are going to switch to clang-format-18 Pull Request resolved: #3372 Reviewed By: kuarora Differential Revision: D56280363 Pulled By: junjieqi fbshipit-source-id: f832ab2112f762e6000b55a155e3e43fe99071d7
1 parent 0169f29 commit 3677ab5

1 file changed

Lines changed: 8 additions & 3 deletions

File tree

.circleci/config.yml

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,12 +38,17 @@ jobs:
3838
- run:
3939
name: Install clang-format
4040
command: |
41-
apt-get update
42-
apt-get install -y git-core clang-format-11
41+
apt-get update -y
42+
apt-get install -y wget
43+
apt install -y lsb-release wget software-properties-common gnupg
44+
wget https://apt.llvm.org/llvm.sh
45+
chmod u+x llvm.sh
46+
./llvm.sh 18
47+
apt-get install -y git-core clang-format-18
4348
- run:
4449
name: Verify clang-format
4550
command: |
46-
git ls-files | grep -E '\.(cpp|h|cu|cuh)$' | xargs clang-format-11 -i
51+
git ls-files | grep -E '\.(cpp|h|cu|cuh)$' | xargs clang-format-18 -i
4752
if git diff --quiet; then
4853
echo "Formatting OK!"
4954
else

0 commit comments

Comments
 (0)