Add a CI check to enforce that files use git lfs when necessary. (#2025)

* Add a CI check to enforce that files use git lfs when necessary. * Remove echo. * Improve error message. * Improve error message. * Update scripts/enforce_git_lfs.sh Co-authored-by: Jake Wharton <[email protected]> * Use bash substring matching. --------- Co-authored-by: Jake Wharton <[email protected]>
cashapp · May 14, 2024 · bc0c23b · bc0c23b
1 parent 644e4fb
commit bc0c23b
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
@@ -24,6 +24,7 @@ jobs:
           java-version: 21
       - uses: gradle/actions/setup-gradle@v3
 
+      - run: ./scripts/enforce_git_lfs.sh
       - run: ./gradlew kotlinUpgradeYarnLock build -PredwoodNoApps
 
   emulator-tests:

diff --git a/scripts/enforce_git_lfs.sh b/scripts/enforce_git_lfs.sh
@@ -0,0 +1,10 @@
+#!/usr/bin/env bash
+
+output=$(git checkout .)
+
+if [[ output == *"files that should have been pointers, but weren't"* ]]; then
+    echo "Error: Encountered files that should have been checked in using Git LFS, but weren't."
+    exit 1
+else
+    exit 0
+fi