Merge pull request #91 from atmina/remove-eslint-plugin-prettier

Remove eslint-plugin-prettier

Author: mvarendorff2

README.md

@@ -3,7 +3,7 @@
 A collection of optionated in-house linting rules.
 
 - ESLint with TypeScript
-- Prettier (via ESLint)
+- Prettier (as of version 2.0, this is [no longer integrated as an ESLint plugin](https://prettier.io/docs/en/integrating-with-linters.html#notes))
 
 ESLint configuration is provided in the `eslint.config.js`, aka. "Flat Config" format.
 

eslint/prettier.js

@@ -1,23 +1,12 @@
-// Runs Prettier as an ESLint rule and reports differences as individual ESLint issues.
-const prettierPlugin = require('eslint-plugin-prettier');
-
 // Turns off all rules that are unnecessary or might conflict with Prettier.
 const prettierOverrides = require('eslint-config-prettier');
 
-const prettierOptions = require('../prettier');
-
 /**
  * @type {import('eslint').Linter.FlatConfig}
  */
 module.exports = {
   files: ['**/*.{ts,tsx,js,jsx}'],
-  plugins: {prettier: prettierPlugin},
   rules: {
     ...prettierOverrides.rules,
-    ...prettierPlugin.configs.recommended.rules,
-    // Note: this effectively locks in our Prettier configuration (ignoring
-    // .prettierrc completely). Since this is an opinionated setup, we probably
-    // don't want to allow customization anyway.
-    'prettier/prettier': ['warn', prettierOptions],
   },
 };

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atmina/linting",
-  "version": "1.1.0",
+  "version": "2.0.0",
   "description": "A collection of opinionated in-house linting rules.",
   "main": "index.js",
   "scripts": {},
@@ -38,7 +38,6 @@
     "eslint-config-prettier": "^9.0.0",
     "eslint-import-resolver-typescript": "^3.5.3",
     "eslint-plugin-import": "^2.27.5",
-    "eslint-plugin-prettier": "^5.0.0-alpha.2",
     "eslint-plugin-react": "^7.32.2",
     "eslint-plugin-react-hooks": "^4.6.0",
     "eslint-plugin-tailwindcss": "^3.10.1",