logical grouping operators differ across shells

yaacovCR · yaacovCR · commit 982e6993814a · 2023-01-13T10:48:47.000+02:00
this isn't perfect, as on my windows bash shell we don't get the error message, but i guess is better, because at least the behavior is correct.

on my machine, it works with curly braces or brackets, but that doesn't work on ci, which seems to want parantheses, which gives the incorrect behavior on my machine!
diff --git a/package.json b/package.json
@@ -16,7 +16,8 @@
     "test": "npm run test:spelling && npm run test:format && npm run test:build",
     "test:spelling": "cspell \"spec/**/*.md\" README.md",
     "format": "prettier --write \"**/*.{md,yml,yaml,json}\"",
-    "test:format": "prettier --check \"**/*.{md,yml,yaml,json}\" || (echo \"\nTo resolve this, run: $(tput bold)npm run format$(tput sgr0)\" && exit 1)",
+    "test:format": "prettier --check \"**/*.{md,yml,yaml,json}\" || npm run suggest:format",
+    "suggest:format": "echo \"\nTo resolve this, run: $(tput bold)npm run format$(tput sgr0)\" && exit 1",
     "build": "./build.sh",
     "test:build": "spec-md --metadata spec/metadata.json spec/GraphQL.md > /dev/null",
     "watch": "nodemon -e json,md --exec \"npm run build\""
