Don't run get_and_print_hashes_of_main unless we need to

open-telemetry · Jun 10, 2024 · 267ed9c · 267ed9c
1 parent e139aa2
commit 267ed9c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/.github/workflows/check-i18n.yml b/.github/workflows/check-i18n.yml
@@ -20,5 +20,5 @@ jobs:
       - name: Any files missing hash key?
         run: scripts/check-i18n.sh -n -x -v
       - name: Any files with invalid hash keys?
-        run: scripts/check-i18n.sh
+        run: scripts/check-i18n.sh -v
       - run: .github/workflows/scripts/check-i18n-helper.sh
diff --git a/scripts/check-i18n.sh b/scripts/check-i18n.sh
@@ -215,8 +215,10 @@ function update_file_i18n_hash() {
 function main() {
   process_CLI_args "$@"
 
-  get_and_print_hashes_of_main
-  if [[ -n $FLAG_INFO ]]; then return; fi
+  if [[ -n $FLAG_INFO ]]; then
+    get_and_print_hashes_of_main
+    return
+  fi
 
   if [ -f "$TARGET_PATHS" ] ; then
     TARGETS="$TARGET_PATHS"
@@ -236,6 +238,7 @@ function main() {
   local FILE_PROCESSED_COUNT=0 # Number of TLP actually listed
 
   if [[ $COMMIT_HASH_ARG == "head" ]]; then
+    if [[ -z $MAIN_HEAD_HASH ]]; then get_and_print_hashes_of_main; fi
     COMMIT_HASH_ARG=$MAIN_HEAD_HASH
   fi