Merge pull request #123 from ChrisCummins/feature/buildifier_122

Format bazel build files.

Author: ChrisCummins

.github/workflows/lint.yaml

@@ -29,3 +29,17 @@ jobs:
         run: python3 -m isort . --check-only --diff --profile black
       - name: black
         run: python3 -m black . --check --diff --target-version py36
+
+  bazel:
+    name: Bazel
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - name: Install go
+        uses: actions/setup-go@v2
+        with:
+          go-version: '^1.13.1'
+      - name: Install buildifier
+        run: go get github.com/bazelbuild/buildtools/buildifier
+      - name: buildifier
+        run: find . \( -name BUILD -o -name WORKSPACE \) -print0 | xargs -0 buildifier -mode=diff

programl/graph/format/BUILD

@@ -111,7 +111,7 @@ cc_test(
         "//programl/proto:program_graph_cc",
         "@labm8//labm8/cpp:status",
         "@labm8//labm8/cpp:test",
-    ]
+    ],
 )
 
 cc_library(

programl/graph/py/BUILD

@@ -36,8 +36,8 @@ py_library(
     data = [":program_graph_builder_pybind.so"],
     visibility = ["//visibility:public"],
     deps = [
-        "//programl/proto:program_graph_py",
         "//programl/proto:program_graph_options_py",
+        "//programl/proto:program_graph_py",
     ],
 )
 
@@ -48,8 +48,8 @@ py_test(
         ":program_graph_builder",
         "//programl/proto:edge_py",
         "//programl/proto:node_py",
-        "//programl/proto:program_graph_py",
         "//programl/proto:program_graph_options_py",
+        "//programl/proto:program_graph_py",
         "//third_party/py/labm8",
     ],
 )

tools/format.sh

@@ -13,6 +13,7 @@ else
     nproc=1
 fi
 
+find . \( -name 'BUILD' -o -name 'WORKSPACE' \) -print0 | xargs -0 buildifier
 find . -name '*.h' -print0 | xargs -0 -n8 -P"$nproc" clang-format -i
 find . -name '*.cc' -print0 | xargs -0 -n8 -P"$nproc" clang-format -i
 find . -name '*.py' -print0 | xargs -0 -n8 -P"$nproc" isort --profile black