commit fa2e26593ec961593c44391e5182a6d57a3918a6
author Matt Johnston <matt@codeconstruct.com.au> Thu Jan 09 16:35:51 2025 +0800
committer Andrew Jeffery <andrew@codeconstruct.com.au> Thu Jan 16 01:28:46 2025 +0000
tree 8fd6c557001d53f341350a91518960679751e073
parent abe9b37f49a1ccda64a728e79858d9eae2d2edd3

scripts: pre-submit: Fail on meson format diff

Explicitly run git diff --exit-code.

When the --quiet argument was removed, the implicit --exit-code was
lost.

gitlint-ignore: UC1
Fixes: 566f2fc1a617 ("scripts: pre-submit: Print meson format diff on failure")
Change-Id: Ia890107b0a8accdf6e1602105487426d6e1ec1b4
Signed-off-by: Matt Johnston <matt@codeconstruct.com.au>

scripts/pre-submit

diff --git a/scripts/pre-submit b/scripts/pre-submit
index dbf990d..59ecace 100755
--- a/scripts/pre-submit
+++ b/scripts/pre-submit
@@ -5,7 +5,7 @@
 trap 'rm -rf "$BUILD"' EXIT
 
 meson format --recursive --inplace || true
-if ! git diff
+if ! git diff --exit-code
 then
   echo Your changes must meet the upstream meson style guide
   echo