Merge branch 'master' into pr/scottohara/826

typescript-eslint · Aug 21, 2019 · e42285d · e42285d
2 parents 2eb3e03 + 6a30de2
commit e42285d
Show file tree

Hide file tree

Showing 183 changed files with 4,963 additions and 605 deletions.
diff --git a/.all-contributorsrc b/.all-contributorsrc
@@ -23,20 +23,20 @@
       "profile": "https://github.com/armano2",
       "contributions": []
     },
-    {
-      "login": "soda0289",
-      "name": "Reyad Attiyat",
-      "avatar_url": "https://avatars1.githubusercontent.com/u/2373964?v=4",
-      "profile": "https://github.com/soda0289",
-      "contributions": []
-    },
     {
       "login": "bradzacher",
       "name": "Brad Zacher",
       "avatar_url": "https://avatars1.githubusercontent.com/u/7462525?v=4",
       "profile": "https://github.com/bradzacher",
       "contributions": []
     },
+    {
+      "login": "soda0289",
+      "name": "Reyad Attiyat",
+      "avatar_url": "https://avatars1.githubusercontent.com/u/2373964?v=4",
+      "profile": "https://github.com/soda0289",
+      "contributions": []
+    },
     {
       "login": "weirdpattern",
       "name": "Patricio Trevino",
@@ -60,7 +60,7 @@
     },
     {
       "login": "uniqueiniquity",
-      "name": "Benjamin Lichtman",
+      "name": "Ben Lichtman",
       "avatar_url": "https://avatars1.githubusercontent.com/u/9092011?v=4",
       "profile": "https://github.com/uniqueiniquity",
       "contributions": []
@@ -79,6 +79,20 @@
       "profile": "https://github.com/Pajn",
       "contributions": []
     },
+    {
+      "login": "mysticatea",
+      "name": "Toru Nagashima",
+      "avatar_url": "https://avatars2.githubusercontent.com/u/1937871?v=4",
+      "profile": "https://github.com/mysticatea",
+      "contributions": []
+    },
+    {
+      "login": "JoshuaKGoldberg",
+      "name": "Josh Goldberg",
+      "avatar_url": "https://avatars1.githubusercontent.com/u/3335181?v=4",
+      "profile": "https://github.com/JoshuaKGoldberg",
+      "contributions": []
+    },
     {
       "login": "azz",
       "name": "Lucas Azzola",
@@ -101,10 +115,10 @@
       "contributions": []
     },
     {
-      "login": "mysticatea",
-      "name": "Toru Nagashima",
-      "avatar_url": "https://avatars2.githubusercontent.com/u/1937871?v=4",
-      "profile": "https://github.com/mysticatea",
+      "login": "scottohara",
+      "name": "Scott O'Hara",
+      "avatar_url": "https://avatars3.githubusercontent.com/u/289327?v=4",
+      "profile": "https://github.com/scottohara",
       "contributions": []
     },
     {
@@ -121,25 +135,67 @@
       "profile": "https://github.com/lukyth",
       "contributions": []
     },
+    {
+      "login": "ldrick",
+      "name": "Ricky Lippmann",
+      "avatar_url": "https://avatars3.githubusercontent.com/u/3674067?v=4",
+      "profile": "https://github.com/ldrick",
+      "contributions": []
+    },
+    {
+      "login": "SimenB",
+      "name": "Simen Bekkhus",
+      "avatar_url": "https://avatars1.githubusercontent.com/u/1404810?v=4",
+      "profile": "https://github.com/SimenB",
+      "contributions": []
+    },
+    {
+      "login": "gavinbarron",
+      "name": "Gavin Barron",
+      "avatar_url": "https://avatars2.githubusercontent.com/u/7122716?v=4",
+      "profile": "https://github.com/gavinbarron",
+      "contributions": []
+    },
     {
       "login": "platinumazure",
       "name": "Kevin Partington",
       "avatar_url": "https://avatars0.githubusercontent.com/u/284282?v=4",
       "profile": "https://github.com/platinumazure",
       "contributions": []
     },
+    {
+      "login": "duailibe",
+      "name": "Lucas Duailibe",
+      "avatar_url": "https://avatars3.githubusercontent.com/u/1574588?v=4",
+      "profile": "https://github.com/duailibe",
+      "contributions": []
+    },
+    {
+      "login": "octogonz",
+      "name": "Pete Gonzalez",
+      "avatar_url": "https://avatars0.githubusercontent.com/u/4673363?v=4",
+      "profile": "https://github.com/octogonz",
+      "contributions": []
+    },
     {
       "login": "mightyiam",
-      "name": "Shahar Or",
+      "name": "Shahar Dawn Or",
       "avatar_url": "https://avatars2.githubusercontent.com/u/635591?v=4",
       "profile": "https://github.com/mightyiam",
       "contributions": []
     },
     {
-      "login": "invalid-email-address",
-      "name": "Check your git settings!",
-      "avatar_url": "https://avatars0.githubusercontent.com/u/148100?v=4",
-      "profile": "https://github.com/invalid-email-address",
+      "login": "a-tarasyuk",
+      "name": "Alexander T.",
+      "avatar_url": "https://avatars0.githubusercontent.com/u/509265?v=4",
+      "profile": "https://github.com/a-tarasyuk",
+      "contributions": []
+    },
+    {
+      "login": "webschik",
+      "name": "Denys Kniazevych",
+      "avatar_url": "https://avatars2.githubusercontent.com/u/1665314?v=4",
+      "profile": "https://github.com/webschik",
       "contributions": []
     },
     {
@@ -155,7 +211,21 @@
       "avatar_url": "https://avatars1.githubusercontent.com/u/17216317?v=4",
       "profile": "https://github.com/g-plane",
       "contributions": []
+    },
+    {
+      "login": "ThomasdenH",
+      "name": "Thomas den Hollander",
+      "avatar_url": "https://avatars0.githubusercontent.com/u/3889750?v=4",
+      "profile": "https://github.com/ThomasdenH",
+      "contributions": []
+    },
+    {
+      "login": "madbence",
+      "name": "Bence Dányi",
+      "avatar_url": "https://avatars2.githubusercontent.com/u/296735?v=4",
+      "profile": "https://github.com/madbence",
+      "contributions": []
     }
   ],
-  "contributorsPerLine": 7
+  "contributorsPerLine": 5
 }
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -15,29 +15,19 @@ module.exports = {
     'eslint:recommended',
     'plugin:@typescript-eslint/eslint-recommended',
     'plugin:@typescript-eslint/recommended',
+    'plugin:@typescript-eslint/recommended-requiring-type-checking',
   ],
   rules: {
     //
     // our plugin :D
     //
 
-    '@typescript-eslint/ban-ts-ignore': 'error',
-    '@typescript-eslint/consistent-type-definitions': 'error',
-    '@typescript-eslint/explicit-function-return-type': 'off',
-    '@typescript-eslint/explicit-member-accessibility': 'off',
-    '@typescript-eslint/indent': 'off',
-    '@typescript-eslint/no-explicit-any': 'warn',
-    '@typescript-eslint/no-inferrable-types': 'error',
-    '@typescript-eslint/no-misused-promises': 'error',
+    '@typescript-eslint/consistent-type-definitions': ['error', 'interface'],
+    '@typescript-eslint/no-explicit-any': 'error',
     '@typescript-eslint/no-non-null-assertion': 'off',
-    '@typescript-eslint/no-object-literal-type-assertion': 'off',
-    '@typescript-eslint/no-parameter-properties': 'off',
-    '@typescript-eslint/no-unnecessary-type-assertion': 'error',
     '@typescript-eslint/no-use-before-define': 'off',
     '@typescript-eslint/no-var-requires': 'off',
-    '@typescript-eslint/prefer-includes': 'error',
-    '@typescript-eslint/prefer-regexp-exec': 'error',
-    '@typescript-eslint/prefer-string-starts-ends-with': 'error',
+    '@typescript-eslint/unbound-method': 'off',
 
     //
     // eslint base
@@ -110,12 +100,12 @@ module.exports = {
     'import/no-mutable-exports': 'error',
     // Prevent importing the default as if it were named
     'import/no-named-default': 'error',
-    // Prohibit named exports // we want everything to be a named export
-    'import/no-named-export': 'off',
+    // Prohibit named exports
+    'import/no-named-export': 'off', // we want everything to be a named export
     // Forbid a module from importing itself
     'import/no-self-import': 'error',
-    // Require modules with a single export to use a default export // we want everything to be named
-    'import/prefer-default-export': 'off',
+    // Require modules with a single export to use a default export
+    'import/prefer-default-export': 'off', // we want everything to be named
   },
   parserOptions: {
     sourceType: 'module',

diff --git a/.gitattributes b/.gitattributes
@@ -1,5 +1,7 @@
 * text=auto
 *.js eol=lf
+*.json eol=lf
+*.md eol=lf
 *.ts eol=lf
 *.tsx eol=lf
 *.yml eol=lf
diff --git a/.github/ISSUE_TEMPLATE.md b/.github/ISSUE_TEMPLATE.md
@@ -0,0 +1,37 @@
+---
+name: catch-all template
+about: Provides some general structure to issues that people choose to log outside the normal flow
+title: ''
+labels: triage
+assignees: ''
+---
+
+**What were you trying to do?**
+
+```json
+{
+  "rules": {
+    "@typescript-eslint/<rule>": ["<setting>"]
+  }
+}
+```
+
+```ts
+// Put your code here
+```
+
+**What did you expect to happen?**
+
+**What actually happened?**
+
+**Versions**
+
+| package                                 | version |
+| --------------------------------------- | ------- |
+| `@typescript-eslint/eslint-plugin`      | `X.Y.Z` |
+| `@typescript-eslint/parser`             | `X.Y.Z` |
+| `@typescript-eslint/typescript-estree`  | `X.Y.Z` |
+| `@typescript-eslint/experimental-utils` | `X.Y.Z` |
+| `TypeScript`                            | `X.Y.Z` |
+| `node`                                  | `X.Y.Z` |
+| `npm`                                   | `X.Y.Z` |
diff --git a/.github/ISSUE_TEMPLATE/eslint-plugin-typescript.md b/.github/ISSUE_TEMPLATE/eslint-plugin-typescript.md
@@ -22,7 +22,7 @@ Are you opening an issue because the rule you're trying to use is not found?
 1) Check the releases log: https://github.com/typescript-eslint/typescript-eslint/releases
     -  If the rule isn't listed there, then chances are it hasn't been released to the main npm tag yet.
 2) Try installing the `canary` tag: `npm i @typescript-eslint/eslint-plugin@canary`.
-    - The canary tag is built for every commit to master, so it contains the bleeding edge build. 
+    - The canary tag is built for every commit to master, so it contains the bleeding edge build.
 3) If ESLint still can't find the rule, then consider reporting an issue.
 -->
 

diff --git a/.github/ISSUE_TEMPLATE/typescript-eslint-utils.md b/.github/ISSUE_TEMPLATE/typescript-eslint-utils.md
@@ -0,0 +1,20 @@
+---
+name: '@typescript-eslint/experimental-utils'
+about: Report an issue with the `@typescript-eslint/experimental-utils` package
+title: ''
+labels: 'package: utils, triage'
+assignees: ''
+---
+
+**What did you expect to happen?**
+
+**What actually happened?**
+
+**Versions**
+
+| package                                 | version |
+| --------------------------------------- | ------- |
+| `@typescript-eslint/experimental-utils` | `X.Y.Z` |
+| `TypeScript`                            | `X.Y.Z` |
+| `node`                                  | `X.Y.Z` |
+| `npm`                                   | `X.Y.Z` |
diff --git a/.gitignore b/.gitignore
@@ -60,3 +60,6 @@ jspm_packages/
 .DS_Store
 .idea
 dist
+
+# Editor-specific metadata folders
+.vs
diff --git a/.prettierignore b/.prettierignore
@@ -9,6 +9,8 @@
 packages/eslint-plugin-tslint/tests/test-tslint-rules-directory/alwaysFailRule.js
 .github
 packages/eslint-plugin/src/configs/*.json
+.all-contributorsrc
+CONTRIBUTORS.md
 
 # Ignore CHANGELOG.md files to avoid issues with automated release job
 CHANGELOG.md
diff --git a/.prettierrc → .prettierrc.json b/.prettierrc → .prettierrc.json
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,72 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [2.0.0](https://github.com/typescript-eslint/typescript-eslint/compare/v1.13.0...v2.0.0) (2019-08-13)
+
+
+* feat(eslint-plugin)!: recommended-requiring-type-checking config (#846) ([d3470c9](https://github.com/typescript-eslint/typescript-eslint/commit/d3470c9)), closes [#846](https://github.com/typescript-eslint/typescript-eslint/issues/846)
+* feat(eslint-plugin)!: change recommended config (#729) ([428567d](https://github.com/typescript-eslint/typescript-eslint/commit/428567d)), closes [#729](https://github.com/typescript-eslint/typescript-eslint/issues/729)
+* feat(typescript-estree)!: throw error on file not in project when `project` set (#760) ([3777b77](https://github.com/typescript-eslint/typescript-eslint/commit/3777b77)), closes [#760](https://github.com/typescript-eslint/typescript-eslint/issues/760)
+* feat(eslint-plugin)!: add rule `consistent-type-assertions` (#731) ([92e98de](https://github.com/typescript-eslint/typescript-eslint/commit/92e98de)), closes [#731](https://github.com/typescript-eslint/typescript-eslint/issues/731)
+* feat(eslint-plugin)!: [array-type] rework options (#654) ([1389393](https://github.com/typescript-eslint/typescript-eslint/commit/1389393)), closes [#654](https://github.com/typescript-eslint/typescript-eslint/issues/654) [#635](https://github.com/typescript-eslint/typescript-eslint/issues/635)
+
+
+### Bug Fixes
+
+* **eslint-plugin:** [efrt] flag default export w/allowExpressions ([#831](https://github.com/typescript-eslint/typescript-eslint/issues/831)) ([ebbcc01](https://github.com/typescript-eslint/typescript-eslint/commit/ebbcc01))
+* **eslint-plugin:** [no-explicit-any] Fix ignoreRestArgs for interfaces ([#777](https://github.com/typescript-eslint/typescript-eslint/issues/777)) ([22e9ae5](https://github.com/typescript-eslint/typescript-eslint/commit/22e9ae5))
+* **eslint-plugin:** [no-useless-constructor] handle bodyless constructor ([#685](https://github.com/typescript-eslint/typescript-eslint/issues/685)) ([55e788c](https://github.com/typescript-eslint/typescript-eslint/commit/55e788c))
+* **eslint-plugin:** [prefer-readonly] TypeError when having comp… ([#761](https://github.com/typescript-eslint/typescript-eslint/issues/761)) ([211b1b5](https://github.com/typescript-eslint/typescript-eslint/commit/211b1b5))
+* **eslint-plugin:** [typedef] support "for..in", "for..of" ([#787](https://github.com/typescript-eslint/typescript-eslint/issues/787)) ([39e41b5](https://github.com/typescript-eslint/typescript-eslint/commit/39e41b5))
+* **eslint-plugin:** [typedef] support default value for parameter ([#785](https://github.com/typescript-eslint/typescript-eslint/issues/785)) ([84916e6](https://github.com/typescript-eslint/typescript-eslint/commit/84916e6))
+* **eslint-plugin:** add `Literal` to `RuleListener` types ([#824](https://github.com/typescript-eslint/typescript-eslint/issues/824)) ([3c902a1](https://github.com/typescript-eslint/typescript-eslint/commit/3c902a1))
+* **typescript-estree:** fix `is` token typed as `Keyword ([#750](https://github.com/typescript-eslint/typescript-eslint/issues/750)) ([35dec52](https://github.com/typescript-eslint/typescript-eslint/commit/35dec52))
+* **typescript-estree:** jsx comment parsing ([#703](https://github.com/typescript-eslint/typescript-eslint/issues/703)) ([0cfc48e](https://github.com/typescript-eslint/typescript-eslint/commit/0cfc48e))
+* **utils:** add ES2019 as valid `ecmaVersion` ([#746](https://github.com/typescript-eslint/typescript-eslint/issues/746)) ([d11fbbe](https://github.com/typescript-eslint/typescript-eslint/commit/d11fbbe))
+
+
+### Features
+
+* explicitly support eslint v6 ([#645](https://github.com/typescript-eslint/typescript-eslint/issues/645)) ([34a7cf6](https://github.com/typescript-eslint/typescript-eslint/commit/34a7cf6))
+* **eslint-plugin:** [interface-name-prefix, class-name-casing] Add allowUnderscorePrefix option to support private declarations ([#790](https://github.com/typescript-eslint/typescript-eslint/issues/790)) ([0c4f474](https://github.com/typescript-eslint/typescript-eslint/commit/0c4f474))
+* **eslint-plugin:** [no-var-requires] report on foo(require('')) ([#725](https://github.com/typescript-eslint/typescript-eslint/issues/725)) ([b2ca20d](https://github.com/typescript-eslint/typescript-eslint/commit/b2ca20d)), closes [#665](https://github.com/typescript-eslint/typescript-eslint/issues/665)
+* **eslint-plugin:** [promise-function-async] make allowAny default true ([#733](https://github.com/typescript-eslint/typescript-eslint/issues/733)) ([590ca50](https://github.com/typescript-eslint/typescript-eslint/commit/590ca50))
+* **eslint-plugin:** [strict-boolean-expressions] add ignoreRhs option ([#691](https://github.com/typescript-eslint/typescript-eslint/issues/691)) ([fd6be42](https://github.com/typescript-eslint/typescript-eslint/commit/fd6be42))
+* **eslint-plugin:** add support for object props in CallExpressions ([#728](https://github.com/typescript-eslint/typescript-eslint/issues/728)) ([8141f01](https://github.com/typescript-eslint/typescript-eslint/commit/8141f01))
+* **eslint-plugin:** added new rule typedef ([#581](https://github.com/typescript-eslint/typescript-eslint/issues/581)) ([35cc99b](https://github.com/typescript-eslint/typescript-eslint/commit/35cc99b))
+* **eslint-plugin:** added new rule use-default-type-parameter ([#562](https://github.com/typescript-eslint/typescript-eslint/issues/562)) ([2b942ba](https://github.com/typescript-eslint/typescript-eslint/commit/2b942ba))
+* **eslint-plugin:** move opinionated rules between configs ([#595](https://github.com/typescript-eslint/typescript-eslint/issues/595)) ([4893aec](https://github.com/typescript-eslint/typescript-eslint/commit/4893aec))
+* **eslint-plugin:** remove deprecated rules ([#739](https://github.com/typescript-eslint/typescript-eslint/issues/739)) ([e32c7ad](https://github.com/typescript-eslint/typescript-eslint/commit/e32c7ad))
+
+
+### BREAKING CHANGES
+
+* removed some rules from recommended config
+* recommended config changes are considered breaking
+* by default we will now throw when a file is not in the `project` provided
+* Merges both no-angle-bracket-type-assertion and no-object-literal-type-assertion into one rule
+* **eslint-plugin:** both 'eslint-recommended' and 'recommended' have changed.
+* **eslint-plugin:** removing rules
+* changes config structure
+
+```ts
+type ArrayOption = 'array' | 'generic' | 'array-simple';
+type Options = [
+  {
+    // default case for all arrays
+    default: ArrayOption,
+    // optional override for readonly arrays
+    readonly?: ArrayOption,
+  },
+];
+```
+* **eslint-plugin:** changing default rule config
+* Node 6 is no longer supported
+
+
+
+
+
 # [1.13.0](https://github.com/typescript-eslint/typescript-eslint/compare/v1.12.0...v1.13.0) (2019-07-21)