Skip to content

Commit cf755a0

Browse files
junjieqiabhinavdangeti
authored andcommitted
Switch clang-format-11 to clang-format-18 (facebookresearch#3372)
Summary: In this commit facebookresearch@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: facebookresearch#3372 Reviewed By: kuarora Differential Revision: D56280363 Pulled By: junjieqi fbshipit-source-id: f832ab2112f762e6000b55a155e3e43fe99071d7
1 parent d61f7c9 commit cf755a0

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

.circleci/config.yml

+8-3
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)