fix variables

Author: khiga8

.github/workflows/axe.yml

@@ -25,7 +25,7 @@ jobs:
             docs/content/utilities/index.md
       - name: Save changed files
         run: |
-          echo "MODIFIED_COMPONENT_OR_UTILITIES_PATHS=${{ steps.changed-files.outputs.all_changed_files }}" >> $GITHUB_ENV
+          echo "CHANGED_DOCS_PATH=(${{ steps.changed-files.outputs.all_changed_files }})" >> $GITHUB_ENV
       - name: Use Node.js
         if: steps.changed-files.outputs.any_changed == 'true'
         uses: actions/setup-node@v3
@@ -42,4 +42,4 @@ jobs:
       - name: Run axe script
         if: steps.changed-files.outputs.any_changed == 'true'
         run: |
-          script/axe-docs $MODIFIED_COMPONENT_OR_UTILITIES_PATHS
+          script/axe-docs $CHANGED_DOCS_PATH

script/axe-docs

@@ -4,10 +4,12 @@
 # Developers frequently copy and paste examples from documentation so it's important to ensure the examples are accessible.
 # Learn about @axe-core/cli: https://github.com/dequelabs/axe-core-npm/tree/develop/packages/cli
 
-array=($MODIFIED_COMPONENT_OR_UTILITIES_PATHS)
-printf "%s\n" "${array[@]}" > temp.txt
+if [ ${#CHANGED_DOCS_PATH[@]} -eq 0 ]; then
+  echo "CHANGED_DOCS_PATH must be set to run axe checks."
+  exit
+fi
 
-# Store the paths of pages we care about
+printf "%s\n" "${CHANGED_DOCS_PATH[@]}" > temp.txt
 nav_paths=()
 while IFS= read -r file
 do
@@ -19,11 +21,7 @@ do
 done < temp.txt
 rm temp.txt
 
-# Return early if no relevant files were modified
-echo "Verify whether ${nav_paths[@]} is part of starting point violation"
-echo "*"
-echo "*"
-
+echo $nav_paths
 # https://stackoverflow.com/a/8574392
 containsElement () {
   local e match="$1"
@@ -32,9 +30,9 @@ containsElement () {
   return 1
 }
 
-# Starting point violations
+# Starting point violations list
 # DO NOT ADD TO THIS LIST!
-skipped=(
+needs_to_be_fixed=(
   /components/autocomplete
   /components/avatars
   /components/box
@@ -56,8 +54,8 @@ skipped=()
 for i in "${nav_paths[@]}"; 
 do
   if containsElement "${i//\"/}" "${needs_to_be_fixed[@]}"; then
+    echo "Skipping $i"
     skipped+=($i)
-    continue
   else
     doc_url="http://localhost:8000$i"
     doc_url="${doc_url//\"/}"
@@ -66,16 +64,18 @@ do
 done
 
 if (( ${#skipped[@]} )); then
-  echo "############################################################"
+  echo "==========================================================================================="
   echo "WARNING"
-  echo "The following paths are excluded from axe checks because they are part of starting point violations:"
-  printf '%s\n' "${needs_to_be_fixed[@]}"
   echo ""
+  echo "Oh no! We had to skip accessibility checks on:"
   echo ""
-  echo "While you're working in:"
   printf '%s\n' "${skipped[@]}"
-  echo "will you consider fixing the accessibility issues so we can remove it from the list? ❤️"
-  echo "############################################################"
+  echo ""
+  echo "because they are part of the starting point violations list."
+  echo ""
+  echo "Help us get one closer to getting rid of this list!"
+  echo "Consider addressing the accessibility issues on the examples of these pages as part of your PR ❤️"
+  echo "==========================================================================================="
 fi
 
 # Run axe checks only if there are pages to run it on