Revert upgrade of Nest.js packages (#332)

* Revert "Publish"

This reverts commit 4405a9b.

* Revert "Upgrade Nest.js on web and CLI packages (#326)"

This reverts commit 4dabf35.

* Revert "Publish"

This reverts commit 4eb6069.

* Revert "Fix common package peer dependency (#328)"

This reverts commit 303d003.

* Revert "Publish"

This reverts commit 6f1c6d8.

* Revert "Revert typedoc upgrade, add table flags to env:list command (#325)"

This reverts commit 327fa71.

* Revert "Upgrade Nest.js and other dependencies (#324)"

This reverts commit 3b210bd.

* Fix electron package on Mac

* Remove jest electron runner

.eslintrc.json

@@ -7,7 +7,7 @@
     "extends": [
         "eslint:recommended",
         "plugin:@typescript-eslint/recommended",
-        "prettier",
+        "prettier/@typescript-eslint",
         "plugin:prettier/recommended",
         "plugin:jest/recommended"
     ],
@@ -18,13 +18,6 @@
     },
     "plugins": ["@typescript-eslint/eslint-plugin"],
     "overrides": [
-        {
-            // We use conditional expect to run tests only in certain platforms.
-            "files": ["packages/common/**/*.ts", "packages/web/**/*.ts", "packages/cli/**/*.ts"],
-            "rules": {
-                "jest/no-conditional-expect": "off"
-            }
-        },
         {
             "files": ["**/*.js"],
             "rules": {
@@ -64,23 +57,13 @@
         {
             "files": ["packages/types/src/**/*.ts"],
             "rules": {
-                "@typescript-eslint/ban-ts-comment": ["off"]
+                "@typescript-eslint/ban-ts-ignore": ["off"]
             }
         }
     ],
     "rules": {
         "@typescript-eslint/consistent-type-assertions": ["error", {"assertionStyle": "never"}],
-        "@typescript-eslint/naming-convention": [
-            "error",
-            {
-                "selector": "interface",
-                "format": ["PascalCase"],
-                "custom": {
-                    "regex": "^I[A-Z]",
-                    "match": true
-                }
-            }
-        ],
+        "@typescript-eslint/interface-name-prefix": ["error", "always"],
         "@typescript-eslint/no-unused-vars": ["error", {"args": "none"}],
         "array-bracket-spacing": ["error", "never"],
         "array-callback-return": "error",
@@ -127,9 +110,7 @@
         "max-statements": "warn",
         "max-statements-per-line": "error",
         "new-parens": "error",
-        // This rule is conflicting with the new behavior from prettier
-        // https://prettier.io/blog/2020/03/21/2.0.0.html#improved-method-chain-breaking-heuristic-6685httpsgithubcomprettierprettierpull6685-by-mmkalhttpsgithubcommmkal
-        "newline-per-chained-call": "off",
+        "newline-per-chained-call": "error",
         "no-alert": "error",
         "no-array-constructor": "error",
         "no-await-in-loop": "error",
@@ -225,9 +206,7 @@
         "no-self-compare": "error",
         "no-sequences": "error",
         "no-setter-return": "error",
-        // The default rule flags all enums as errors
-        "no-shadow": "off",
-        "@typescript-eslint/no-shadow": ["error"],
+        "no-shadow": "error",
         "no-sync": "error",
         "no-tabs": "error",
         "no-template-curly-in-string": "error",

.nvmrc

@@ -1 +1 @@
-14
+12

.prettierrc

@@ -1,11 +1,11 @@
 {
-    "printWidth": 120,
-    "semi": true,
-    "singleQuote": true,
-    "useTabs": false,
-    "tabWidth": 4,
-    "arrowParens": "always",
-    "trailingComma": "all",
-    "bracketSpacing": false,
-    "endOfLine": "lf"
+  "printWidth": 120,
+  "semi": true,
+  "singleQuote": true,
+  "useTabs": false,
+  "tabWidth": 4,
+  "arrowParens": "always",
+  "trailingComma": "all",
+  "bracketSpacing": false,
+  "endOfLine": "lf"
 }