ci: Improve tidiness check

Avoid fragile change directory with a sub-shell and improve documentation.
RamenDR · Nov 22, 2024 · 01892af · 01892af
1 parent 319b99a
commit 01892af
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/.github/workflows/ramen.yaml b/.github/workflows/ramen.yaml
@@ -54,16 +54,15 @@ jobs:
       - name: Run linters
         run: ./hack/pre-commit.sh
 
-      - name: Check that generated files were not modified
+      - name: Check tidiness
+        # Check that modules are tidy, and that generated files were not
+        # modified by the pull request. Both validated by empty diff.
         run: |
           make generate
           make manifests
           go mod tidy
-          cd ./e2e/
-          go mod tidy
-          cd ../api
-          go mod tidy
-          cd ..
+          (cd e2e && go mod tidy)
+          (cd api && go mod tidy)
           git --no-pager diff
           git diff-index --quiet HEAD