enable require-meta-docs-description rule (#620)

Author: Dimitri POSTOLOV

.eslintrc.js

@@ -2,13 +2,7 @@ module.exports = {
   reportUnusedDisableDirectives: true,
   ignorePatterns: ['examples'],
   parser: '@typescript-eslint/parser',
-  extends: [
-    'eslint:recommended',
-    'plugin:@typescript-eslint/recommended',
-    'standard',
-    'plugin:eslint-plugin/recommended',
-    'prettier',
-  ],
+  extends: ['eslint:recommended', 'plugin:@typescript-eslint/recommended', 'standard', 'prettier'],
   plugins: ['unicorn'],
   rules: {
     'no-empty': 'off',
@@ -29,19 +23,30 @@ module.exports = {
     '@typescript-eslint/ban-types': 'off',
     'unicorn/prefer-array-some': 'error',
     'unicorn/prefer-includes': 'error',
-    'eslint-plugin/test-case-shorthand-strings': 'error',
-    'eslint-plugin/no-only-tests': 'error',
-    'eslint-plugin/require-meta-docs-url': [
-      'error',
-      { pattern: 'https://github.com/dotansimha/graphql-eslint/blob/master/docs/rules/{{name}}.md' },
-    ],
   },
   overrides: [
+    {
+      files: ['**/rules/*.ts'],
+      extends: ['plugin:eslint-plugin/rules-recommended'],
+      rules: {
+        'eslint-plugin/no-deprecated-context-methods': 'error',
+        'eslint-plugin/require-meta-docs-description': ['error', { pattern: '.+\\.' }], // force to put a point at the end
+        'eslint-plugin/require-meta-docs-url': [
+          'error',
+          { pattern: 'https://github.com/dotansimha/graphql-eslint/blob/master/docs/rules/{{name}}.md' },
+        ],
+      },
+    },
     {
       files: ['*.{spec,test}.{ts,js}'],
       env: {
         jest: true,
       },
+      extends: ['plugin:eslint-plugin/tests-recommended'],
+      rules: {
+        'eslint-plugin/no-only-tests': 'error',
+        'eslint-plugin/test-case-shorthand-strings': 'error',
+      },
     },
     {
       files: ['**/tests/mocks/*.{ts,js}'],

docs/README.md

@@ -24,10 +24,10 @@ Each rule has emojis denoting:
 | [known-type-names](rules/known-type-names.md) | A GraphQL document is only valid if referenced types (specifically variable definitions and fragment conditions) are defined by the type schema. |     🔮 |  |
 | [lone-anonymous-operation](rules/lone-anonymous-operation.md) | A GraphQL document is only valid if when it contains an anonymous operation (the query short-hand) that it contains only that one operation definition. |     🔮 |  |
 | [lone-schema-definition](rules/lone-schema-definition.md) | A GraphQL document is only valid if it contains only one schema definition. |     🔮 |  |
-| [match-document-filename](rules/match-document-filename.md) | This rule allows you to enforce that the file name should match the operation name |     🚀 |  |
+| [match-document-filename](rules/match-document-filename.md) | This rule allows you to enforce that the file name should match the operation name. |     🚀 |  |
 | [naming-convention](rules/naming-convention.md) | Require names to follow specified conventions. |     🚀 |  |
 | [no-anonymous-operations](rules/no-anonymous-operations.md) | Require name for your GraphQL operations. This is useful since most GraphQL client libraries are using the operation name for caching purposes. |     🚀 |  |
-| [no-case-insensitive-enum-values-duplicates](rules/no-case-insensitive-enum-values-duplicates.md) |  |     🚀 | 🔧 |
+| [no-case-insensitive-enum-values-duplicates](rules/no-case-insensitive-enum-values-duplicates.md) | Disallow case-insensitive enum values duplicates. |     🚀 | 🔧 |
 | [no-deprecated](rules/no-deprecated.md) | Enforce that deprecated fields or enum values are not in use by operations. |     🚀 |  |
 | [no-fragment-cycles](rules/no-fragment-cycles.md) | A GraphQL fragment is only valid when it does not have cycles in fragments usage. |     🔮 |  |
 | [no-hashtag-description](rules/no-hashtag-description.md) | Requires to use `"""` or `"` for adding a GraphQL description instead of `#`. |     🚀 |  |

docs/rules/match-document-filename.md

@@ -5,7 +5,7 @@
 - Requires GraphQL Schema: `false` [ℹ️](../../README.md#extended-linting-rules-with-graphql-schema)
 - Requires GraphQL Operations: `false` [ℹ️](../../README.md#extended-linting-rules-with-siblings-operations)
 
-This rule allows you to enforce that the file name should match the operation name
+This rule allows you to enforce that the file name should match the operation name.
 
 ## Usage Examples
 

docs/rules/no-case-insensitive-enum-values-duplicates.md

@@ -5,7 +5,7 @@
 - Requires GraphQL Schema: `false` [ℹ️](../../README.md#extended-linting-rules-with-graphql-schema)
 - Requires GraphQL Operations: `false` [ℹ️](../../README.md#extended-linting-rules-with-siblings-operations)
 
-
+Disallow case-insensitive enum values duplicates.
 
 ## Usage Examples
 

packages/plugin/src/rules/match-document-filename.ts

@@ -41,7 +41,7 @@ const rule: GraphQLESLintRule<MatchDocumentFilenameRuleConfig> = {
     type: 'suggestion',
     docs: {
       category: 'Best Practices',
-      description: 'This rule allows you to enforce that the file name should match the operation name',
+      description: 'This rule allows you to enforce that the file name should match the operation name.',
       url: `https://github.com/dotansimha/graphql-eslint/blob/master/docs/rules/match-document-filename.md`,
       examples: [
         {

packages/plugin/src/rules/no-case-insensitive-enum-values-duplicates.ts

@@ -10,6 +10,7 @@ const rule: GraphQLESLintRule = {
         'https://github.com/dotansimha/graphql-eslint/blob/master/docs/rules/no-case-insensitive-enum-values-duplicates.md',
       category: 'Best Practices',
       recommended: true,
+      description: 'Disallow case-insensitive enum values duplicates.',
       examples: [
         {
           title: 'Incorrect',