From 1c3db8e18ebf50d95c26727255cc187b11d7c0eb Mon Sep 17 00:00:00 2001
From: Vince Grassia <593223+vgrassia@users.noreply.github.com>
Date: Thu, 13 May 2021 16:43:19 -0400
Subject: [PATCH] Update locales test in build workflow

---
 .github/workflows/build.yml | 34 +++++++++++++++++++++-------------
 1 file changed, 21 insertions(+), 13 deletions(-)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index b71702bd85..3fdb2f43b2 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -50,19 +50,27 @@ jobs:
 
       - name: Testing locales - extName length
         run: |
-          exit_status=0
-          
-          echo "extName string must be 40 characters or less"
-          echo "===================================="
-          for locale in $(ls src/_locales/); do
-            string_length=$(jq '.extName.message | length' src/_locales/$locale/messages.json)
-            if [[ $string_length -gt 40 ]]; then
-              echo $locale: $string_length
-              exit_status=1
-            fi
-          done
-          
-          exit $exit_status
+         found_error=false
+
+         echo "Locales Test"
+         echo "============"
+         echo "extName string must be 40 characters or less"
+         echo
+         for locale in $(ls src/_locales/); do
+           string_length=$(jq '.extName.message | length' src/_locales/$locale/messages.json)
+           if [[ $string_length -gt 40 ]]; then
+             echo "$locale: $string_length"
+             found_error=true
+           fi
+         done
+         
+         if $found_error; then
+           echo
+           echo "Please fix 'extName' for the locales listed above."
+           exit 1
+         else
+           echo "Test passed!"
+         fi 
 
   build:
     runs-on: windows-latest