Editorial: If no prose changes prettier workflow should not fail (#2641)

Author: daniel-montalvo

.github/workflows/prettier.yaml

@@ -31,15 +31,30 @@ jobs:
           changed_files=$(git diff --name-only HEAD~1 HEAD | grep -E '\.(js|json|css|html)$' || true)
 
           if [[ -n "$changed_files" ]]; then
+            echo "Found changed files to format:"
+            echo "$changed_files"
+            
+            # Run prettier on the changed files
             echo "$changed_files" | xargs prettier --write --print-width 200
-      
-            echo "changed=true" >> "$GITHUB_OUTPUT"
+            
+            # Check if prettier actually made any changes
+            if git diff --quiet && git diff --cached --quiet; then
+              echo "No formatting changes were made by Prettier"
+              echo "prettier_changes=false" >> "$GITHUB_OUTPUT"
+            else
+              echo "Prettier made formatting changes"
+              echo "prettier_changes=true" >> "$GITHUB_OUTPUT"
+            fi
+            
+            echo "prose_changes=true" >> "$GITHUB_OUTPUT"
           else
-            echo "changed=false" >> "$GITHUB_OUTPUT"
+            echo "No JavaScript, JSON, CSS, or HTML files were changed in this commit."
+            echo "prose_changes=false" >> "$GITHUB_OUTPUT"
+            exit 0
           fi
 
       - name: Commit and push changes
-        if: steps.run_prettier.outputs.changed == 'true'
+        if: steps.run_prettier.outputs.prettier_changes == 'true'
         run: |
           git config --local user.name "github-actions[bot]"
           git config --local user.email "github-actions[bot]@users.noreply.github.com"
@@ -49,7 +64,4 @@ jobs:
 
           # Push to the main branch using PAT
           git push origin HEAD:${{ github.ref_name }}
-
-      - name: No changes to commit
-        if: steps.run_prettier.outputs.changed == 'false'
-        run: echo "No changes to commit."
+          echo "Changes committed and pushed successfully"