Merge remote-tracking branch 'origin/master' into strict-type-predicates

typescript-eslint · Sep 8, 2019 · f68b732 · f68b732
2 parents 6d37bf4 + f1059d8
commit f68b732
Show file tree

Hide file tree

Showing 230 changed files with 10,478 additions and 2,861 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,20 @@ 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/explicit-function-return-type': 'error',
+    '@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 +101,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
@@ -1,4 +1,5 @@
 **/tests/fixtures/**/*
+**/tests/fixture-project/**/*
 **/dist
 **/coverage
 **/shared-fixtures
@@ -8,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