浏览代码

Switch clang-format pre-commit to get 13.0.0 (#1066)

Also enable gh action, I'm expecting that to work now.
Jon Meow 4 年之前
父节点
当前提交
58104dc364
共有 3 个文件被更改,包括 8 次插入8 次删除
  1. 1 2
      .github/workflows/pre-commit.yaml
  2. 4 2
      .pre-commit-config.yaml
  3. 3 4
      toolchain/parser/parser_impl.cpp

+ 1 - 2
.github/workflows/pre-commit.yaml

@@ -18,5 +18,4 @@ jobs:
       - uses: pre-commit/action@v2.0.2
         env:
           # bazel-buildifier relies on a locally installed copy.
-          # clang-format-whole-file has races in cache creation.
-          SKIP: bazel-buildifier,clang-format-whole-file
+          SKIP: bazel-buildifier

+ 4 - 2
.pre-commit-config.yaml

@@ -60,8 +60,10 @@ repos:
     rev: fb5e481f35f5a9b32a06ccfc28b8d6dda4525f75
     hooks:
       - id: bazel-buildifier
-      - id: clang-format-whole-file
-        types: [c++]
+  - repo: https://github.com/pre-commit/mirrors-clang-format
+    rev: 1fc50313b6e8c2580c4736af57575e0b7de1501c # frozen: v13.0.0
+    hooks:
+      - id: clang-format
   - repo: https://github.com/pre-commit/mirrors-prettier
     rev: ea782651a7e32f40a3d13b76c79d5a2474ee8723 # frozen: v2.5.1
     hooks:

+ 3 - 4
toolchain/parser/parser_impl.cpp

@@ -804,10 +804,9 @@ auto ParseTree::Parser::ParseBraceExpression() -> llvm::Optional<Node> {
         }
 
         // Work out the kind of this element
-        Kind elem_kind =
-            (NextTokenIs(TokenKind::Equal())
-                 ? Value
-                 : NextTokenIs(TokenKind::Colon()) ? Type : Unknown);
+        Kind elem_kind = (NextTokenIs(TokenKind::Equal())   ? Value
+                          : NextTokenIs(TokenKind::Colon()) ? Type
+                                                            : Unknown);
         if (elem_kind == Unknown || (kind != Unknown && elem_kind != kind)) {
           return diagnose_invalid_syntax();
         }