Merge pull request 'format check shows green check, not anymore when format shows fail' (#110) from format-check-rust-fix into main

Reviewed-on: #110 Reviewed-by: zach <zach@brohn.se>
2026-05-29 11:50:10 +02:00
parent 428809fee8 0c47ba805f
commit 0819e8e2d1
1 changed files with 3 additions and 1 deletions
@@ -42,11 +42,13 @@ jobs:
              continue-on-error: true
              run: |
                  if [ -n "$(find . -name "Cargo.toml" -print -quit)" ]; then
+                    status=0
                    for manifest in $(find . -name "Cargo.toml"); do
                      cargo fmt --manifest-path "$manifest" --check && \
                        echo "$manifest: formatting OK" || \
-                        echo "$manifest: needs formatting"
+                        { echo "$manifest: needs formatting"; status=1; }
                    done
+                    exit $status
                  elif [ -n "$(find . -name "*.rs" -print -quit)" ]; then
                    echo "Rust files found but no Cargo.toml"
                    exit 1