Merge pull request #80 from NullVoxPopuli/do-not-allow-warnings

Do not allow warnings in our tests

Author: NullVoxPopuli

package.json

@@ -20,10 +20,10 @@
   "scripts": {
     "test": "vitest run",
     "lint": "concurrently \"npm:lint:*(!fix)\" --names \"lint:\"",
-    "lint:js": "eslint .",
+    "lint:js": "eslint . --max-warnings=0",
     "lint:package": "pnpm publint",
     "lint:fix": "concurrently \"npm:lint:*:fix\" --names \"fix:\"",
-    "lint:js:fix": "eslint . --fix"
+    "lint:js:fix": "eslint . --fix --max-warnings=0"
   },
   "devDependencies": {
     "@babel/core": "^7.23.6",

test-projects/configs/flat-js/package.json

@@ -4,7 +4,7 @@
   "scripts": {
     "test:notes": "This isn't just 'eslint .' because we aren't in a type=module package",
     "test": "pnpm eslint:with-config .",
-    "eslint:with-config": "ESLINT_USE_FLAT_CONFIG=true eslint --config ./eslint.config.mjs",
+    "eslint:with-config": "ESLINT_USE_FLAT_CONFIG=true eslint --config ./eslint.config.mjs --max-warnings=0",
     "eslint:debug-file": "pnpm eslint:with-config --print-config"
   },
   "devDependencies": {

test-projects/gjs/package.json

@@ -2,7 +2,7 @@
   "name": "@test-project/gjs",
   "private": true,
   "scripts": {
-    "test": "eslint ."
+    "test": "eslint . --max-warnings=0"
   },
   "devDependencies": {
     "@typescript-eslint/eslint-plugin": "^6.21.0",

test-projects/gts/package.json

@@ -2,7 +2,7 @@
   "name": "@test-project/gts",
   "private": true,
   "scripts": {
-    "test": "eslint .",
+    "test": "eslint . --max-warnings=0",
     "types": "tsc --noEmit"
   },
   "devDependencies": {

test-projects/rules/padding-line-between-statements/package.json

@@ -2,7 +2,7 @@
   "name": "@test-project/rules-padding-line-between-statements",
   "private": true,
   "scripts": {
-    "test": "eslint ."
+    "test": "eslint . --max-warnings=0"
   },
   "devDependencies": {
     "@typescript-eslint/eslint-plugin": "^6.21.0",