feat: remove overrides from jest and typescript configs

BREAKING CHANGE: jest and typescript configs will now need to be explicitly set up to apply only to certain files

Author: yenbekbay

README.md

@@ -49,6 +49,21 @@ module.exports = {
 };
 ```
 
+You can also use [ESLint@4 overrides](http://eslint.org/docs/user-guide/configuring#configuration-based-on-glob-patterns) to apply a config only to certain files. For example:
+
+```js
+module.exports = {
+  ...
+  overrides: [
+    Object.assign(
+      {
+        files: ['**/__tests__/*-test.js', '**/__mocks__/*.js'],
+      },
+      require('eslint-config-anvilabs/jest')
+    ),
+  ],
+};
+
 Available configs include:
 
 - `'anvilabs/babel'` for usage with [babel transformations](https://github.com/babel/babel-eslint)

jest.js

@@ -1,17 +1,12 @@
 module.exports = {
-  overrides: [
-    {
-      plugins: ['jest'],
-      files: '**/*-test.{js,ts}',
-      env: {
-        'jest/globals': true,
-      },
-      rules: {
-        'jest/no-disabled-tests': 'warn',
-        'jest/no-focused-tests': 'error',
-        'jest/no-identical-title': 'error',
-        'jest/valid-expect': 'error',
-      },
-    },
-  ],
+  plugins: ['jest'],
+  env: {
+    'jest/globals': true,
+  },
+  rules: {
+    'jest/no-disabled-tests': 'warn',
+    'jest/no-focused-tests': 'error',
+    'jest/no-identical-title': 'error',
+    'jest/valid-expect': 'error',
+  },
 };

typescript.js

@@ -1,74 +1,69 @@
 module.exports = {
-  overrides: [
-    {
-      files: '**/*.ts',
-      plugins: ['typescript'],
-      parser: 'typescript-eslint-parser',
-      settings: {
-        'import/resolver': {
-          node: {
-            extensions: ['.js', '.ts', '.json'],
-          },
+  plugins: ['typescript'],
+  parser: 'typescript-eslint-parser',
+  settings: {
+    'import/resolver': {
+      node: {
+        extensions: ['.js', '.ts', '.json'],
+      },
+    },
+    'import/extensions': ['.js', '.ts'],
+  },
+  rules: {
+    'no-undef': 'off', // conflicts with typescript
+    'no-unused-vars': 'off', // conflicts with typescript
+    'no-useless-constructor': 'off', // conflicts with typescript
+    'spaced-comment': [
+      'error',
+      'always',
+      {
+        line: {
+          exceptions: ['-', '+'],
+          markers: ['=', '!', '/'], // space here to support sprockets directives
+        },
+        block: {
+          exceptions: ['-', '+'],
+          markers: ['=', '!'], // space here to support sprockets directives
+          balanced: true,
         },
-        'import/extensions': ['.js', '.ts'],
       },
-      rules: {
-        'no-undef': 'off', // conflicts with typescript
-        'no-unused-vars': 'off', // conflicts with typescript
-        'no-useless-constructor': 'off', // conflicts with typescript
-        'spaced-comment': [
-          'error',
-          'always',
-          {
-            line: {
-              exceptions: ['-', '+'],
-              markers: ['=', '!', '/'], // space here to support sprockets directives
-            },
-            block: {
-              exceptions: ['-', '+'],
-              markers: ['=', '!'], // space here to support sprockets directives
-              balanced: true,
-            },
-          },
-        ],
-        strict: 'off', // conflicts with typescript
-        // https://github.com/benmosher/eslint-plugin-import
-        'import/extensions': [
-          'error',
-          'always',
-          {
-            js: 'never',
-            ts: 'never',
-          },
-        ],
-        'import/no-named-as-default-member': 'off', // conflicts with typescript
-        'import/no-named-as-default': 'off', // conflicts with typescript
-        // https://github.com/not-an-aardvark/eslint-plugin-prettier
-        'prettier/prettier': [
-          'error',
-          {
-            useTabs: false,
-            printWidth: 80,
-            tabWidth: 2,
-            singleQuote: true,
-            trailingComma: 'all',
-            bracketSpacing: false,
-            jsxBracketSameLine: false,
-            parser: 'typescript',
-            semi: true,
-          },
-        ],
-        // https://github.com/nzakas/eslint-plugin-typescript
-        'typescript/explicit-member-accessibility': 'error',
-        'typescript/interface-name-prefix': 'error',
-        'typescript/no-angle-bracket-type-assertion': 'error',
-        'typescript/no-explicit-any': 'error',
-        'typescript/no-namespace': ['error', {allowDefinitionFiles: true}],
-        'typescript/no-triple-slash-reference': 'error',
-        'typescript/no-unused-vars': 'error',
-        'typescript/prefer-namespace-keyword': 'error',
-        'typescript/type-annotation-spacing': 'off',
+    ],
+    strict: 'off', // conflicts with typescript
+    // https://github.com/benmosher/eslint-plugin-import
+    'import/extensions': [
+      'error',
+      'always',
+      {
+        js: 'never',
+        ts: 'never',
       },
-    },
-  ],
+    ],
+    'import/no-named-as-default-member': 'off', // conflicts with typescript
+    'import/no-named-as-default': 'off', // conflicts with typescript
+    // https://github.com/not-an-aardvark/eslint-plugin-prettier
+    'prettier/prettier': [
+      'error',
+      {
+        useTabs: false,
+        printWidth: 80,
+        tabWidth: 2,
+        singleQuote: true,
+        trailingComma: 'all',
+        bracketSpacing: false,
+        jsxBracketSameLine: false,
+        parser: 'typescript',
+        semi: true,
+      },
+    ],
+    // https://github.com/nzakas/eslint-plugin-typescript
+    'typescript/explicit-member-accessibility': 'error',
+    'typescript/interface-name-prefix': 'error',
+    'typescript/no-angle-bracket-type-assertion': 'error',
+    'typescript/no-explicit-any': 'error',
+    'typescript/no-namespace': ['error', {allowDefinitionFiles: true}],
+    'typescript/no-triple-slash-reference': 'error',
+    'typescript/no-unused-vars': 'error',
+    'typescript/prefer-namespace-keyword': 'error',
+    'typescript/type-annotation-spacing': 'off',
+  },
 };