[CHORE] switched to prettier-eslint and added vscode config (#182)

Added config for the vscode prettier extension because I got sick of manually formatting and relying on the on commit formatter..

Also switched to `prettier-eslint` so that the eslint fixers get auto-run by the format process.

Author: bradzacher

.editorconfig

@@ -5,4 +5,4 @@ indent_style = space
 indent_size = 4
 trim_trailing_whitespace = true
 end_of_line = lf
-insert_final_newline = true
+insert_final_newline = true

.eslintrc

@@ -0,0 +1,21 @@
+{
+    "root": true,
+    "plugins": [
+        "node",
+        "prettier"
+    ],
+    "extends": [
+        "eslint",
+        "plugin:node/recommended",
+        "prettier"
+    ],
+    "rules": {
+        "prettier/prettier": [
+            "error",
+            {
+                "tabWidth": 4
+            }
+        ],
+        "lines-around-comment": "off"
+    }
+}

.eslintrc.yml

@@ -12,6 +12,5 @@ versions.json
 .eslintcache
 .cache
 /packages/**/node_modules
-/.vscode
 .sublimelinterrc
 yarn.lock

.gitignore

@@ -0,0 +1 @@
+node_modules

.prettierignore

@@ -0,0 +1,11 @@
+{
+    "arrowParens": "avoid",
+    "bracketSpacing": true,
+    "jsxBracketSameLine": true,
+    "proseWrap": "preserve",
+    "semi": true,
+    "singleQuote": false,
+    "tabWidth": 4,
+    "trailingComma": "es5",
+    "useTabs": false
+}

.prettierrc

@@ -0,0 +1,38 @@
+{
+    // Format a file on save. A formatter must be available, the file must not be auto-saved, and editor must not be shutting down.
+    "editor.formatOnSave": true,
+
+    // An array of languages where Emmet abbreviations should not be expanded.
+    "emmet.excludeLanguages": [
+        "markdown",
+        "typescript"
+    ],
+
+    // An array of language ids which should be validated by ESLint
+    "eslint.validate": [
+        "javascript",
+        "javascriptreact",
+        {
+            "language": "typescript",
+            "autoFix": true
+        },
+        {
+            "language": "typescriptreact",
+            "autoFix": true
+        }
+    ],
+
+    // When enabled, will trim trailing whitespace when saving a file.
+    "files.trimTrailingWhitespace": true,
+
+    // Use 'prettier-eslint' instead of 'prettier'. Other settings will only be fallbacks in case they could not be inferred from eslint rules.
+    "prettier.eslintIntegration": true,
+
+    // typescript auto-format settings
+    "typescript.tsdk": "node_modules/typescript/lib",
+    "javascript.preferences.importModuleSpecifier": "auto",
+    "typescript.preferences.importModuleSpecifier": "auto",
+    "javascript.preferences.quoteStyle": "double",
+    "typescript.preferences.quoteStyle": "double",
+    "typescript.updateImportsOnFileMove.enabled": "always",
+}

.vscode/settings.json

@@ -14,9 +14,9 @@ module.exports = {
             description: "Require that member overloads be consecutive",
             category: "TypeScript",
             url:
-                "https://github.com/nzakas/eslint-plugin-typescript/blob/master/docs/rules/adjacent-overload-signatures.md"
+                "https://github.com/nzakas/eslint-plugin-typescript/blob/master/docs/rules/adjacent-overload-signatures.md",
         },
-        schema: []
+        schema: [],
     },
 
     create(context) {
@@ -92,7 +92,7 @@ module.exports = {
                     if (index > -1 && lastName !== name) {
                         context.report({
                             node: member,
-                            message: `All '${name}' signatures should be adjacent`
+                            message: `All '${name}' signatures should be adjacent`,
                         });
                     } else if (name && index === -1) {
                         seen.push(name);
@@ -111,7 +111,7 @@ module.exports = {
             TSTypeLiteral: checkBodyForOverloadMethods,
             TSInterfaceBody: checkBodyForOverloadMethods,
             ClassBody: checkBodyForOverloadMethods,
-            Program: checkBodyForOverloadMethods
+            Program: checkBodyForOverloadMethods,
         };
-    }
+    },
 };

lib/rules/adjacent-overload-signatures.js

@@ -18,8 +18,8 @@ module.exports = {
             category: "Best Practices",
             recommended: true,
             url:
-                "https://github.com/nzakas/eslint-plugin-typescript/blob/master/docs/rules/class-name-casing.md"
-        }
+                "https://github.com/nzakas/eslint-plugin-typescript/blob/master/docs/rules/class-name-casing.md",
+        },
     },
 
     create(context) {
@@ -63,7 +63,7 @@ module.exports = {
 
             context.report({
                 node: id,
-                message: `${friendlyName} '${id.name}' must be PascalCased`
+                message: `${friendlyName} '${id.name}' must be PascalCased`,
             });
         }
 
@@ -88,7 +88,7 @@ module.exports = {
                         report(node.init, id);
                     }
                 }
-            }
+            },
         };
-    }
+    },
 };

lib/rules/class-name-casing.js

@@ -18,19 +18,19 @@ module.exports = {
 
             category: "TypeScript",
             url:
-                "https://github.com/nzakas/eslint-plugin-typescript/blob/master/docs/rules/explicit-function-return-type.md"
+                "https://github.com/nzakas/eslint-plugin-typescript/blob/master/docs/rules/explicit-function-return-type.md",
         },
         schema: [
             {
                 type: "object",
                 properties: {
                     allowExpressions: {
-                        type: "boolean"
-                    }
+                        type: "boolean",
+                    },
                 },
-                additionalProperties: false
-            }
-        ]
+                additionalProperties: false,
+            },
+        ],
     },
 
     create(context) {
@@ -86,7 +86,7 @@ module.exports = {
             ) {
                 context.report({
                     node,
-                    message: `Missing return type on function.`
+                    message: `Missing return type on function.`,
                 });
             }
         }
@@ -97,7 +97,7 @@ module.exports = {
         return {
             FunctionDeclaration: checkFunctionReturnType,
             FunctionExpression: checkFunctionReturnType,
-            ArrowFunctionExpression: checkFunctionReturnType
+            ArrowFunctionExpression: checkFunctionReturnType,
         };
-    }
+    },
 };