Bug 1677562 - Remove eslint-plugin-prettier now that Prettier and ESLint are separated. r=Gijs,devtools-reviewers,ochameau

Depends on D175769

Differential Revision: https://phabricator.services.mozilla.com/D175770

browser/components/newtab/.eslintrc.js

@@ -112,8 +112,6 @@ module.exports = {
     },
   ],
   rules: {
-    "prettier/prettier": "off",
-
     "fetch-options/no-fetch-credentials": "error",
 
     "react/jsx-boolean-value": ["error", "always"],

devtools/client/debugger/src/.eslintrc.js

@@ -3,7 +3,7 @@
  * file, You can obtain one at <http://mozilla.org/MPL/2.0/>. */
 
 module.exports = {
-  plugins: ["react", "mozilla", "@babel", "prettier", "import", "file-header"],
+  plugins: ["react", "mozilla", "@babel", "import", "file-header"],
   globals: {
     atob: true,
     btoa: true,
@@ -47,8 +47,6 @@ module.exports = {
     jest: true,
   },
   rules: {
-    "prettier/prettier": "off",
-
     // These are the rules that have been configured so far to match the
     // devtools coding style.
 

package.json

@@ -4,28 +4,27 @@
   "repository": {},
   "license": "MPL-2.0",
   "devDependencies": {
-    "@babel/core": "7.20.12",
+    "@babel/core": "7.21.4",
-    "@babel/eslint-parser": "7.19.1",
+    "@babel/eslint-parser": "7.21.3",
     "@babel/eslint-plugin": "7.19.1",
-    "@babel/plugin-syntax-jsx": "7.18.6",
+    "@babel/plugin-syntax-jsx": "7.21.4",
     "@microsoft/eslint-plugin-sdl": "github:mozfreddyb/eslint-plugin-sdl#17b22cd527682108af7a1a4edacf69cb7a9b4a06",
-    "eslint": "8.33.0",
+    "eslint": "8.38.0",
-    "eslint-config-prettier": "8.6.0",
+    "eslint-config-prettier": "8.8.0",
     "eslint-plugin-eslint-plugin": "5.0.8",
     "eslint-plugin-fetch-options": "0.0.5",
     "eslint-plugin-file-header": "0.0.1",
     "eslint-plugin-html": "7.1.0",
     "eslint-plugin-import": "2.27.5",
     "eslint-plugin-jest": "23.20.0",
-    "eslint-plugin-jsdoc": "39.7.4",
+    "eslint-plugin-jsdoc": "39.9.1",
     "eslint-plugin-jsx-a11y": "6.7.1",
     "eslint-plugin-mozilla": "file:tools/lint/eslint/eslint-plugin-mozilla",
     "eslint-plugin-no-unsanitized": "4.0.2",
-    "eslint-plugin-prettier": "3.4.0",
     "eslint-plugin-react": "7.29.4",
     "eslint-plugin-react-hooks": "4.6.0",
     "eslint-plugin-spidermonkey-js": "file:tools/lint/eslint/eslint-plugin-spidermonkey-js",
-    "jsdoc": "4.0.0",
+    "jsdoc": "4.0.2",
     "prettier": "1.19.1",
     "yarn": "1.22.19"
   },

tools/lint/eslint/eslint-plugin-mozilla/lib/configs/recommended.js

@@ -113,7 +113,7 @@ module.exports = {
   },
 
   // When adding items to this file please check for effects on sub-directories.
-  plugins: ["html", "fetch-options", "prettier", "no-unsanitized"],
+  plugins: ["html", "fetch-options", "no-unsanitized"],
 
   // When adding items to this file please check for effects on all of toolkit
   // and browser
@@ -339,8 +339,6 @@ module.exports = {
     // This may conflict with prettier, so turn it off.
     "prefer-arrow-callback": "off",
 
-    "prettier/prettier": "off",
-
     // This generates too many false positives that are not easy to work around,
     // and false positives seem to be inherent in the rule.
     "require-atomic-updates": "off",

tools/lint/eslint/eslint-plugin-mozilla/manifest.tt

@@ -1,9 +1,9 @@
 [
   {
     "filename": "eslint-plugin-mozilla.tar.gz",
-    "size": 11475311,
+    "size": 9278498,
     "algorithm": "sha512",
-    "digest": "309cab6d8a66533505decc708d1f0a6f65c1a639c4c2c740b0c0649d766efe8206c0b51abaa59ae990c201dc039ac8b0295987f2862ac5b568b4e8e9341300ca",
+    "digest": "01615c6a29c0027fabe528d9a6e2133b80467e0e0c420bab3a282dea3c8de1fd59c44dab0fcdb19c09e72c7d0d7e81347029336e77b4463a8c09272876da10a8",
     "unpack": true,
     "visibility": "public"
   }

tools/lint/eslint/eslint-plugin-mozilla/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-mozilla",
-  "version": "3.0.2",
+  "version": "3.1.0",
   "description": "A collection of rules that help enforce JavaScript coding standard in the Mozilla project.",
   "keywords": [
     "eslint",
@@ -29,7 +29,7 @@
     "multi-ini": "^2.3.2"
   },
   "devDependencies": {
-    "eslint": "8.33.0",
+    "eslint": "8.38.0",
     "mocha": "10.2.0"
   },
   "peerDependencies": {
@@ -38,9 +38,7 @@
     "eslint-config-prettier": "^8.0.0",
     "eslint-plugin-fetch-options": "^0.0.5",
     "eslint-plugin-html": "^7.0.0",
-    "eslint-plugin-no-unsanitized": "^4.0.0",
+    "eslint-plugin-no-unsanitized": "^4.0.0"
-    "eslint-plugin-prettier": "^3.0.0",
-    "prettier": "^1.19.1"
   },
   "engines": {
     "node": ">=14.0.0"

tools/lint/eslint/manifest.tt

@@ -1,9 +1,9 @@
 [
   {
     "filename": "eslint.tar.gz",
-    "size": 28539167,
+    "size": 24096013,
     "algorithm": "sha512",
-    "digest": "25d3154ec47678bf1943822a446dd3c4e775f5df58c0197daef0d5da807b9912f6d7bbde8f3bf538d57057b0b9412df00275cddea8b040a8e163490fe157ab93",
+    "digest": "dcf615e3d47219b6e6e7eeef9f72559bf0c59d0c503ec3d4e784cf0f2ad0178d6dac9341486ec6cb8dcfbaf02718e93470888ff7e8a3c16ca3171514f4b8136f",
     "unpack": true,
     "visibility": "public"
   }