feat: remove prettier rule, add related docs (#397)

* feat: remove `prettier` rule, add related docs

* docs: add changeset for the feat change

Author: JounQin

.changeset/chilly-foxes-help.md

@@ -0,0 +1,5 @@
+---
+"@graphql-eslint/eslint-plugin": minor
+---
+
+feat: remove `prettier` rule, add related docs

README.md

@@ -169,6 +169,55 @@ You can find a complete list of [all available rules here](./docs/README.md)
 
 > This repo doesn't exports a "recommended" set of rules - feel free to recommend us!
 
+### `prettier` rule
+
+The original `prettier` rule has been removed because `eslint-plugin-prettier` supports `.graphql` files well actually.
+
+All you need to do is like the following for now:
+
+```js
+// .eslintrc.js
+module.exports = {
+  overrides: [
+    {
+      files: ['*.tsx', '*.ts', '*.jsx', '*.js'],
+      processor: '@graphql-eslint/graphql',
+    },
+    {
+      files: ['*.graphql'],
+      parser: '@graphql-eslint/eslint-plugin',
+      plugins: ['@graphql-eslint'],
+      // the following is required for `eslint-plugin-prettier@<=3.4.0` temporarily
+      // after https://github.com/prettier/eslint-plugin-prettier/pull/413
+      // been merged and released, it can be deleted safely
+      rules: {
+        'prettier/prettier': [
+          2,
+          {
+            parser: 'graphql',
+          },
+        ],
+      },
+    },
+    // the following is required for `eslint-plugin-prettier@<=3.4.0` temporarily
+    // after https://github.com/prettier/eslint-plugin-prettier/pull/415
+    // been merged and released, it can be deleted safely
+    {
+      files: ['*.js/*.graphql'],
+      rules: {
+        'prettier/prettier': 0
+      }
+    },
+  ],
+};
+```
+
+You can take [`examples/prettier`](examples/prettier/.eslintrc.js) as example.
+
+It could be better to remove the unnecessary parser setting if https://github.com/prettier/eslint-plugin-prettier/pull/413 and https://github.com/prettier/eslint-plugin-prettier/pull/415 been merged and released.
+
+Please help to vote up if you want to speed up the progress.
+
 ## Further Reading
 
 If you wish to learn more about this project, how the parser works, how to add custom rules and more, [please refer to the docs directory](./docs/README.md))

docs/README.md

@@ -20,7 +20,6 @@
 - [`naming-convention`](./rules/naming-convention.md)
 - [`input-name`](./rules/input-name.md)
 - [`strict-id-in-types`](./rules/strict-id-in-types.md)
-- [`prettier`](./rules/prettier.md)
 - [`executable-definitions`](./rules/executable-definitions.md)
 - [`fields-on-correct-type`](./rules/fields-on-correct-type.md)
 - [`fragments-on-composite-type`](./rules/fragments-on-composite-type.md)

docs/rules/prettier.md

@@ -9,9 +9,9 @@
     "lint": "eslint --ext graphql,js ."
   },
   "dependencies": {
-    "eslint": "7.14.0",
+    "eslint": "7.24.0",
     "@graphql-eslint/eslint-plugin": "0.9.1",
-    "graphql": "15.4.0",
-    "typescript": "4.1.2"
+    "graphql": "15.5.0",
+    "typescript": "4.1.5"
   }
 }

examples/basic/package.json

@@ -9,9 +9,9 @@
     "lint": "eslint --ext graphql,js ."
   },
   "dependencies": {
-    "eslint": "7.14.0",
+    "eslint": "7.24.0",
     "@graphql-eslint/eslint-plugin": "0.9.1",
-    "graphql": "15.4.0",
-    "typescript": "4.1.2"
+    "graphql": "15.5.0",
+    "typescript": "4.1.5"
   }
 }

examples/code-file/package.json

@@ -10,9 +10,9 @@
   },
   "dependencies": {
     "@graphql-eslint/eslint-plugin": "0.9.1",
-    "eslint": "7.14.0",
-    "graphql": "15.4.0",
+    "eslint": "7.24.0",
+    "graphql": "15.5.0",
     "graphql-tag": "^2.11.0",
-    "typescript": "4.1.2"
+    "typescript": "4.1.5"
   }
 }

examples/graphql-config-code-file/package.json

@@ -10,9 +10,9 @@
   },
   "dependencies": {
     "graphql-config": "3.2.0",
-    "eslint": "7.14.0",
+    "eslint": "7.24.0",
     "@graphql-eslint/eslint-plugin": "0.9.1",
-    "graphql": "15.4.0",
-    "typescript": "4.1.2"
+    "graphql": "15.5.0",
+    "typescript": "4.1.5"
   }
 }

examples/graphql-config/package.json

@@ -0,0 +1,34 @@
+module.exports = {
+  extends: ['plugin:prettier/recommended'],
+  overrides: [
+    {
+      files: ['*.tsx', '*.ts', '*.jsx', '*.js'],
+      processor: '@graphql-eslint/graphql',
+    },
+    {
+      files: ['*.graphql'],
+      parser: '@graphql-eslint/eslint-plugin',
+      plugins: ['@graphql-eslint'],
+      // the following is required for `eslint-plugin-prettier@<=3.4.0` temporarily
+      // after https://github.com/prettier/eslint-plugin-prettier/pull/413
+      // been merged and released, it can be deleted safely
+      rules: {
+        'prettier/prettier': [
+          2,
+          {
+            parser: 'graphql',
+          },
+        ],
+      },
+    },
+    // the following is required for `eslint-plugin-prettier@<=3.4.0` temporarily
+    // after https://github.com/prettier/eslint-plugin-prettier/pull/415
+    // been merged and released, it can be deleted safely
+    {
+      files: ['*.js/*.graphql'],
+      rules: {
+        'prettier/prettier': 0,
+      },
+    },
+  ],
+};

examples/prettier/.eslintrc.js

@@ -0,0 +1,16 @@
+query GetUser($userId:      ID!) {
+  user(id: $userId) {
+        id,
+  name,
+    isViewerFriend,
+    profilePicture(size: 50)  {
+      ...PictureFragment
+    }
+  }
+}
+
+fragment PictureFragment on Picture {
+  uri,
+  width,
+  height
+}