ACS-8920 Fix pre-commit for empty commits (#1035)

* ACS-8920 Test empty commit * ACS-8920 Test empty commit * ACS-8920 Test empty commit * ACS-8920 Test empty commit * ACS-8920 Pre-commit job should fail * Revert "ACS-8920 Pre-commit job should fail" This reverts commit 80516b7. * ACS-8920 Test empty commit * ACS-8920 Remove temporary condition
Alfresco · Oct 29, 2024 · 22abe07 · 22abe07
1 parent 39bc461
commit 22abe07
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -37,11 +37,6 @@ env:
 jobs:
   pre_commit:
     runs-on: ubuntu-latest
-    if: >
-      !contains(github.event.head_commit.message, '[release]')
-    outputs:
-      java_version: ${{ env.JAVA_VERSION }}
-      java_version_supported_by_repo: ${{ env.JAVA_VERSION_SUPPORTED_BY_REPO }}
     steps:
       - uses: Alfresco/alfresco-build-tools/.github/actions/[email protected]
       - uses: actions/checkout@v4

diff --git a/scripts/check-format-and-headers.sh b/scripts/check-format-and-headers.sh
@@ -7,6 +7,12 @@ else
   modified_files=${GITHUB_MODIFIED_FILES}
 fi
 
+if [[ -z ${modified_files} ]]
+then
+  echo "No modified files, exiting."
+  exit 0
+fi
+
 include_list=""
 for file in ${modified_files}
 do