ci: Simplify git diff checks

Signed-off-by: provokateurin <[email protected]>
nextcloud · Oct 7, 2024 · 1ab8160 · 1ab8160
1 parent e70c0d4
commit 1ab8160
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 21 deletions.
diff --git a/.github/workflows/dart.yml b/.github/workflows/dart.yml
@@ -36,11 +36,7 @@ jobs:
         run: ./tool/setup.sh
 
       - name: Check up-to-date pubspec.lock
-        run: |
-          if [ -n "$(git status --porcelain)" ]; then
-            git --no-pager diff
-            exit 1
-          fi
+        run: git diff --no-pager --exit-code
       - name: Check formatting
         run: melos run format:check
       - name: Lint code

diff --git a/.github/workflows/generate.yaml b/.github/workflows/generate.yaml
@@ -48,26 +48,17 @@ jobs:
       - name: Generate dynamite end to end test
         run: |
           ./tool/generate-dynamite-e2e-test.sh
-          if [ -n "$(git status --porcelain)" ]; then
-            git --no-pager diff
-            exit 1
-          fi
+          git diff --no-pager --exit-code
 
       - name: Generate dynamite petstore example
         run: |
           ./tool/generate-dynamite-petstore-example.sh
-          if [ -n "$(git status --porcelain)" ]; then
-            git --no-pager diff
-            exit 1
-          fi
+          git diff --no-pager --exit-code
 
       - name: Generate nextcloud
         run: |
           ./tool/generate-nextcloud.sh
-          if [ -n "$(git status --porcelain)" ]; then
-            git --no-pager diff
-            exit 1
-          fi
+          git diff --no-pager --exit-code
 
   openapi:
     name: OpenAPI
@@ -95,7 +86,4 @@ jobs:
       - name: Generate specs
         run: |
           ./tool/generate-specs.sh
-          if [ -n "$(git status --porcelain)" ]; then
-            git --no-pager diff
-            exit 1
-          fi
+          git diff --no-pager --exit-code