Add check-json to pre-commit (#4792)

b/393208250
youtube · Jan 30, 2025 · 5e27148 · 5e27148
1 parent 0b34129
commit 5e27148
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/.github/actions/build/action.yaml b/.github/actions/build/action.yaml
@@ -53,15 +53,15 @@ runs:
           # TODO(b/382508397): Replace hardcoded list with dynamically generated one.
           TEST_TARGETS_JSON_FILE: cobalt/build/testing/targets/${{ matrix.platform }}/test_targets.json
         run: |
-          set -x
+          set -ex
           cd src
           time autoninja -C out/${{ matrix.platform }}_${{ matrix.config }} $(cat "${TEST_TARGETS_JSON_FILE}" | jq -cr '.test_targets | join(" ")')
         shell: bash
       - name: Ninja build
         env:
           TARGETS: ${{ inputs.targets }}
         run: |
-          set -x
+          set -ex
           cd src
           time autoninja -C out/${{ matrix.platform }}_${{ matrix.config }} ${TARGETS}
         shell: bash

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -27,6 +27,7 @@ repos:
     -   id: check-case-conflict
     -   id: check-xml
     -   id: check-yaml
+    -   id: check-json
     -   id: end-of-file-fixer
     -   id: mixed-line-ending
     -   id: trailing-whitespace