Fail CI on ESLint warnings (#2807)

dpwatrous · web-flow · commit 637891aa682e · 2023-09-27T10:20:19.000+08:00
diff --git a/.prettierignore b/.prettierignore
@@ -5,6 +5,7 @@
 /**/test-results/
 /**/bin/
 /**/build/
+/**/generated/
 
 *.md
 *.html
@@ -19,4 +20,3 @@ package-lock.json
 Localize/out/
 Localize/packages/
 /**/resources/
-packages/arm-batch-rest/src/generated/
diff --git a/packages/arm-batch-rest/package.json b/packages/arm-batch-rest/package.json
@@ -40,7 +40,7 @@
     "clean:esm": "bux rmrf ./lib",
     "clean:cjs": "bux rmrf ./lib-cjs",
     "generate:client": "autorest --typescript swagger/README.md",
-    "lint": "eslint",
+    "lint": "eslint . --max-warnings 0",
     "test": "jest",
     "test:all": "npm run test:coverage",
     "test:coverage": "jest --collect-coverage",
diff --git a/packages/bonito-core/package.json b/packages/bonito-core/package.json
@@ -31,7 +31,7 @@
     "test:all": "npm run test:coverage",
     "test:debug": "node --inspect-brk ./node_modules/.bin/jest --runInBand --watch",
     "test:watch": "jest --watch",
-    "lint": "eslint .",
+    "lint": "eslint . --max-warnings 0",
     "watch": "run-p \"compile:* -- --watch --preserveWatchOutput\"",
     "workspace:build": "npm run build",
     "workspace:build:clean": "npm run build:clean",
diff --git a/packages/bonito-ui/package.json b/packages/bonito-ui/package.json
@@ -33,7 +33,7 @@
     "test:watch": "jest --watch",
     "test:a11y": "cross-env BE_ENABLE_A11Y_TESTING=true npm run test",
     "test:a11y:watch": "cross-env BE_ENABLE_A11Y_TESTING=true npm run test:watch",
-    "lint": "eslint .",
+    "lint": "eslint . --max-warnings 0",
     "watch": "run-p \"compile:* -- --watch --preserveWatchOutput\"",
     "workspace:build": "npm run build",
     "workspace:build:clean": "npm run build:clean",
diff --git a/packages/playground/package.json b/packages/playground/package.json
@@ -32,7 +32,7 @@
     "test:all": "cross-env BE_ENABLE_A11Y_TESTING=true npm run test:coverage",
     "test:debug": "node --inspect-brk ./node_modules/.bin/jest --runInBand --watch",
     "test:watch": "jest --watch",
-    "lint": "eslint .",
+    "lint": "eslint . --max-warnings 0",
     "watch": "run-p \"compile:* -- --watch --preserveWatchOutput\"",
     "workspace:watch:lib": "npm run watch",
     "workspace:build": "npm run build",
diff --git a/packages/react/package.json b/packages/react/package.json
@@ -34,7 +34,7 @@
     "test:watch": "jest --watch",
     "test:a11y": "cross-env BE_ENABLE_A11Y_TESTING=true npm run test",
     "test:a11y:watch": "cross-env BE_ENABLE_A11Y_TESTING=true npm run test:watch",
-    "lint": "eslint .",
+    "lint": "eslint . --max-warnings 0",
     "watch": "run-p \"compile:* -- --watch --preserveWatchOutput\"",
     "workspace:build": "npm run build",
     "workspace:build:clean": "npm run build:clean",
diff --git a/packages/service/package.json b/packages/service/package.json
@@ -31,7 +31,7 @@
     "test:all": "npm run test:coverage",
     "test:debug": "node --inspect-brk ./node_modules/.bin/jest --runInBand --watch",
     "test:watch": "jest --watch",
-    "lint": "eslint .",
+    "lint": "eslint . --max-warnings 0",
     "watch": "run-p \"compile:* -- --watch --preserveWatchOutput\"",
     "workspace:build": "npm run build",
     "workspace:build:clean": "npm run build:clean",
diff --git a/util/bux/package.json b/util/bux/package.json
@@ -20,7 +20,7 @@
     "build": "tsc -b ./config/tsconfig.build.json && node ./bin/bux chmodx ./bin/bux",
     "bux": "npm run build && node ./bin/bux",
     "clean": "rimraf ./lib && rimraf ./build",
-    "lint": "eslint .",
+    "lint": "eslint . --max-warnings 0",
     "lint:fix": "npm run lint --fix",
     "build:clean": "run-s clean build",
     "prepack": "npm run build:clean",
diff --git a/util/common-config/.eslintrc-common.json b/util/common-config/.eslintrc-common.json
@@ -14,6 +14,7 @@
   ],
   "ignorePatterns": [
     "**/coverage/*",
+    "**/generated/*",
     "**/lib/**/*",
     "**/lib-*/**/*",
     "**/node_modules/**/*"
diff --git a/web/package.json b/web/package.json
@@ -36,7 +36,7 @@
     "test:all": "cross-env BE_ENABLE_A11Y_TESTING=true npm run test:coverage",
     "test:debug": "node --inspect-brk ./node_modules/.bin/jest --runInBand --watch",
     "test:watch": "jest --watch",
-    "lint": "eslint .",
+    "lint": "eslint . --max-warnings 0",
     "server": "webpack-dev-server",
     "start": "npm run start:dev",
     "start:dev": "npm run clean && npm run build-translations && npm run server -- --env dev",
