Sort out eslint

shapkarin · May 18, 2021 · e2caf9b · e2caf9b
1 parent 5392482
commit e2caf9b
Show file tree

Hide file tree

Showing 59 changed files with 3,095 additions and 59 deletions.
diff --git a/.eslintrc.base.js b/.eslintrc.base.js
@@ -0,0 +1,92 @@
+module.exports = {
+  env: {
+    es6: true,
+    jasmine: true,
+    node: true,
+    worker: true,
+  },
+  parser: "@typescript-eslint/parser",
+  parserOptions: {
+    ecmaVersion: 2018,
+    project: "./tsconfig.eslint.json",
+    tsconfigRootDir: __dirname,
+  },
+  plugins: ["@typescript-eslint", "prettier", "simple-import-sort", "import"],
+  extends: [
+    "eslint:recommended",
+    "plugin:@typescript-eslint/recommended",
+    "prettier",
+    "plugin:prettier/recommended",
+    "plugin:import/typescript",
+  ],
+  rules: {
+    curly: ["warn", "multi-line", "consistent"],
+    "no-bitwise": "warn",
+    "no-console": ["warn", { allow: ["error", "info", "table", "warn"] }],
+    "no-param-reassign": "warn",
+    "no-shadow": "off", // disabled in favour of @typescript-eslint/no-shadow, see https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-shadow.md
+    "no-unused-vars": "off", // disabled in favour of @typescript-eslint/no-unused-vars, see https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-unused-vars.md
+    "prefer-const": "warn",
+    radix: ["warn", "always"],
+    "spaced-comment": ["warn", "always", { line: { markers: ["/ <reference"] } }],
+    "import/no-cycle": "warn",
+    "simple-import-sort/imports": "warn",
+    "@typescript-eslint/array-type": ["warn", { default: "array-simple" }],
+    "@typescript-eslint/await-thenable": "warn",
+    "@typescript-eslint/ban-types": "warn",
+    "@typescript-eslint/explicit-function-return-type": ["warn", { allowExpressions: true }],
+    "@typescript-eslint/explicit-member-accessibility": "warn",
+    "@typescript-eslint/naming-convention": [
+      "warn",
+      {
+        selector: "default",
+        format: ["strictCamelCase"],
+      },
+      {
+        selector: "typeLike",
+        format: ["StrictPascalCase"],
+      },
+      {
+        selector: "enumMember",
+        format: ["StrictPascalCase"],
+      },
+      {
+        selector: "variable",
+        format: ["strictCamelCase"],
+        leadingUnderscore: "allow",
+      },
+      {
+        selector: "parameter",
+        format: ["strictCamelCase"],
+        leadingUnderscore: "allow",
+      },
+    ],
+    "@typescript-eslint/no-dynamic-delete": "warn",
+    "@typescript-eslint/no-empty-function": "off",
+    "@typescript-eslint/no-empty-interface": "off",
+    "@typescript-eslint/no-explicit-any": "off",
+    "@typescript-eslint/no-floating-promises": "warn",
+    "@typescript-eslint/no-parameter-properties": "warn",
+    "@typescript-eslint/no-shadow": "warn",
+    "@typescript-eslint/no-unused-vars": ["warn", { argsIgnorePattern: "^_", varsIgnorePattern: "^_" }],
+    "@typescript-eslint/no-unnecessary-type-assertion": "warn",
+    "@typescript-eslint/no-use-before-define": "warn",
+    "@typescript-eslint/prefer-readonly": "warn",
+  },
+  overrides: [
+    {
+      files: "**/*.js",
+      rules: {
+        "@typescript-eslint/no-var-requires": "off",
+        "@typescript-eslint/explicit-function-return-type": "off",
+        "@typescript-eslint/explicit-member-accessibility": "off",
+      },
+    },
+    {
+      files: "**/*.spec.ts",
+      rules: {
+        "@typescript-eslint/no-non-null-assertion": "off",
+      },
+    },
+  ],
+};
diff --git a/.eslintrc.js → packages/amino/.eslintrc.js b/.eslintrc.js → packages/amino/.eslintrc.js
@@ -8,14 +8,14 @@ module.exports = {
   parser: "@typescript-eslint/parser",
   parserOptions: {
     ecmaVersion: 2018,
-    project: "./tsconfig.json",
+    project: "./tsconfig.eslint.json",
     tsconfigRootDir: __dirname,
   },
   plugins: ["@typescript-eslint", "prettier", "simple-import-sort", "import"],
   extends: [
     "eslint:recommended",
     "plugin:@typescript-eslint/recommended",
-    "prettier/@typescript-eslint",
+    "prettier",
     "plugin:prettier/recommended",
     "plugin:import/typescript",
   ],
@@ -30,7 +30,7 @@ module.exports = {
     radix: ["warn", "always"],
     "spaced-comment": ["warn", "always", { line: { markers: ["/ <reference"] } }],
     "import/no-cycle": "warn",
-    "simple-import-sort/sort": "warn",
+    "simple-import-sort/imports": "warn",
     "@typescript-eslint/array-type": ["warn", { default: "array-simple" }],
     "@typescript-eslint/await-thenable": "warn",
     "@typescript-eslint/ban-types": "warn",

diff --git a/packages/amino/package.json b/packages/amino/package.json
@@ -23,8 +23,8 @@
   },
   "scripts": {
     "docs": "typedoc --options typedoc.js",
-    "lint": "eslint --max-warnings 0 \"**/*.{js,ts}\"",
-    "lint-fix": "eslint --max-warnings 0 \"**/*.{js,ts}\" --fix",
+    "lint": "eslint --max-warnings 0 \"./**/*.ts\" \"./*.js\"",
+    "lint-fix": "eslint --fix --max-warnings 0 \"./**/*.ts\" \"./*.js\"",
     "format": "prettier --write --loglevel warn \"./src/**/*.ts\"",
     "format-text": "prettier --write \"./*.md\"",
     "test-node": "node jasmine-testrunner.js",
@@ -46,9 +46,17 @@
   },
   "devDependencies": {
     "@istanbuljs/nyc-config-typescript": "^1.0.1",
+    "@types/eslint-plugin-prettier": "^3",
     "@types/jasmine": "^3.6.10",
+    "@typescript-eslint/eslint-plugin": "^4.22.1",
+    "@typescript-eslint/parser": "^4.22.1",
     "crypto-browserify": "^3.12.0",
     "eslint": "^7.5",
+    "eslint-config-prettier": "^8.3.0",
+    "eslint-import-resolver-node": "^0.3.4",
+    "eslint-plugin-import": "^2.22.1",
+    "eslint-plugin-prettier": "^3.4.0",
+    "eslint-plugin-simple-import-sort": "^7.0.0",
     "glob": "^7.1.6",
     "jasmine": "^3.5",
     "jasmine-spec-reporter": "^6",

diff --git a/packages/amino/tsconfig.eslint.json b/packages/amino/tsconfig.eslint.json
@@ -0,0 +1,9 @@
+{
+  // extend your base config so you don't have to redefine your compilerOptions
+  "extends": "./tsconfig.json",
+  "include": [
+    "src/**/*",
+    "*.js",
+    ".eslintrc.js"
+  ]
+}
diff --git a/packages/cli/.eslintrc.js b/packages/cli/.eslintrc.js
@@ -0,0 +1,92 @@
+module.exports = {
+  env: {
+    es6: true,
+    jasmine: true,
+    node: true,
+    worker: true,
+  },
+  parser: "@typescript-eslint/parser",
+  parserOptions: {
+    ecmaVersion: 2018,
+    project: "./tsconfig.eslint.json",
+    tsconfigRootDir: __dirname,
+  },
+  plugins: ["@typescript-eslint", "prettier", "simple-import-sort", "import"],
+  extends: [
+    "eslint:recommended",
+    "plugin:@typescript-eslint/recommended",
+    "prettier",
+    "plugin:prettier/recommended",
+    "plugin:import/typescript",
+  ],
+  rules: {
+    curly: ["warn", "multi-line", "consistent"],
+    "no-bitwise": "warn",
+    "no-console": ["warn", { allow: ["error", "info", "table", "warn"] }],
+    "no-param-reassign": "warn",
+    "no-shadow": "off", // disabled in favour of @typescript-eslint/no-shadow, see https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-shadow.md
+    "no-unused-vars": "off", // disabled in favour of @typescript-eslint/no-unused-vars, see https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-unused-vars.md
+    "prefer-const": "warn",
+    radix: ["warn", "always"],
+    "spaced-comment": ["warn", "always", { line: { markers: ["/ <reference"] } }],
+    "import/no-cycle": "warn",
+    "simple-import-sort/imports": "warn",
+    "@typescript-eslint/array-type": ["warn", { default: "array-simple" }],
+    "@typescript-eslint/await-thenable": "warn",
+    "@typescript-eslint/ban-types": "warn",
+    "@typescript-eslint/explicit-function-return-type": ["warn", { allowExpressions: true }],
+    "@typescript-eslint/explicit-member-accessibility": "warn",
+    "@typescript-eslint/naming-convention": [
+      "warn",
+      {
+        selector: "default",
+        format: ["strictCamelCase"],
+      },
+      {
+        selector: "typeLike",
+        format: ["StrictPascalCase"],
+      },
+      {
+        selector: "enumMember",
+        format: ["StrictPascalCase"],
+      },
+      {
+        selector: "variable",
+        format: ["strictCamelCase"],
+        leadingUnderscore: "allow",
+      },
+      {
+        selector: "parameter",
+        format: ["strictCamelCase"],
+        leadingUnderscore: "allow",
+      },
+    ],
+    "@typescript-eslint/no-dynamic-delete": "warn",
+    "@typescript-eslint/no-empty-function": "off",
+    "@typescript-eslint/no-empty-interface": "off",
+    "@typescript-eslint/no-explicit-any": "off",
+    "@typescript-eslint/no-floating-promises": "warn",
+    "@typescript-eslint/no-parameter-properties": "warn",
+    "@typescript-eslint/no-shadow": "warn",
+    "@typescript-eslint/no-unused-vars": ["warn", { argsIgnorePattern: "^_", varsIgnorePattern: "^_" }],
+    "@typescript-eslint/no-unnecessary-type-assertion": "warn",
+    "@typescript-eslint/no-use-before-define": "warn",
+    "@typescript-eslint/prefer-readonly": "warn",
+  },
+  overrides: [
+    {
+      files: "**/*.js",
+      rules: {
+        "@typescript-eslint/no-var-requires": "off",
+        "@typescript-eslint/explicit-function-return-type": "off",
+        "@typescript-eslint/explicit-member-accessibility": "off",
+      },
+    },
+    {
+      files: "**/*.spec.ts",
+      rules: {
+        "@typescript-eslint/no-non-null-assertion": "off",
+      },
+    },
+  ],
+};
diff --git a/packages/cli/package.json b/packages/cli/package.json
@@ -17,8 +17,8 @@
   "scripts": {
     "format": "prettier --write --loglevel warn \"./src/**/*.ts\"",
     "format-text": "prettier --write \"./*.md\"",
-    "lint": "eslint --max-warnings 0 \"**/*.{js,ts}\"",
-    "lint-fix": "eslint --max-warnings 0 \"**/*.{js,ts}\" --fix",
+    "lint": "eslint --max-warnings 0 \"./**/*.ts\" \"./*.js\"",
+    "lint-fix": "eslint --fix --max-warnings 0 \"./**/*.ts\" \"./*.js\"",
     "build": "rm -rf ./build && tsc",
     "build-or-skip": "[ -n \"$SKIP_BUILD\" ] || yarn build",
     "start": "yarn build-or-skip && ./bin/cosmwasm-cli",
@@ -63,10 +63,18 @@
     "@istanbuljs/nyc-config-typescript": "^1.0.1",
     "@types/babylon": "^6.16.3",
     "@types/diff": "^4",
+    "@types/eslint-plugin-prettier": "^3",
     "@types/jasmine": "^3.6.10",
     "@types/node": "^15.0.1",
     "@types/yargs": "^15.0.4",
+    "@typescript-eslint/eslint-plugin": "^4.22.1",
+    "@typescript-eslint/parser": "^4.22.1",
     "eslint": "^7.5",
+    "eslint-config-prettier": "^8.3.0",
+    "eslint-import-resolver-node": "^0.3.4",
+    "eslint-plugin-import": "^2.22.1",
+    "eslint-plugin-prettier": "^3.4.0",
+    "eslint-plugin-simple-import-sort": "^7.0.0",
     "jasmine": "^3.5",
     "jasmine-spec-reporter": "^6",
     "nyc": "^15.1.0",

diff --git a/packages/cli/tsconfig.eslint.json b/packages/cli/tsconfig.eslint.json
@@ -0,0 +1,9 @@
+{
+  // extend your base config so you don't have to redefine your compilerOptions
+  "extends": "./tsconfig.json",
+  "include": [
+    "src/**/*",
+    "*.js",
+    ".eslintrc.js"
+  ]
+}
diff --git a/packages/cosmwasm-launchpad/.eslintrc.js b/packages/cosmwasm-launchpad/.eslintrc.js
@@ -0,0 +1,92 @@
+module.exports = {
+  env: {
+    es6: true,
+    jasmine: true,
+    node: true,
+    worker: true,
+  },
+  parser: "@typescript-eslint/parser",
+  parserOptions: {
+    ecmaVersion: 2018,
+    project: "./tsconfig.eslint.json",
+    tsconfigRootDir: __dirname,
+  },
+  plugins: ["@typescript-eslint", "prettier", "simple-import-sort", "import"],
+  extends: [
+    "eslint:recommended",
+    "plugin:@typescript-eslint/recommended",
+    "prettier",
+    "plugin:prettier/recommended",
+    "plugin:import/typescript",
+  ],
+  rules: {
+    curly: ["warn", "multi-line", "consistent"],
+    "no-bitwise": "warn",
+    "no-console": ["warn", { allow: ["error", "info", "table", "warn"] }],
+    "no-param-reassign": "warn",
+    "no-shadow": "off", // disabled in favour of @typescript-eslint/no-shadow, see https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-shadow.md
+    "no-unused-vars": "off", // disabled in favour of @typescript-eslint/no-unused-vars, see https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-unused-vars.md
+    "prefer-const": "warn",
+    radix: ["warn", "always"],
+    "spaced-comment": ["warn", "always", { line: { markers: ["/ <reference"] } }],
+    "import/no-cycle": "warn",
+    "simple-import-sort/imports": "warn",
+    "@typescript-eslint/array-type": ["warn", { default: "array-simple" }],
+    "@typescript-eslint/await-thenable": "warn",
+    "@typescript-eslint/ban-types": "warn",
+    "@typescript-eslint/explicit-function-return-type": ["warn", { allowExpressions: true }],
+    "@typescript-eslint/explicit-member-accessibility": "warn",
+    "@typescript-eslint/naming-convention": [
+      "warn",
+      {
+        selector: "default",
+        format: ["strictCamelCase"],
+      },
+      {
+        selector: "typeLike",
+        format: ["StrictPascalCase"],
+      },
+      {
+        selector: "enumMember",
+        format: ["StrictPascalCase"],
+      },
+      {
+        selector: "variable",
+        format: ["strictCamelCase"],
+        leadingUnderscore: "allow",
+      },
+      {
+        selector: "parameter",
+        format: ["strictCamelCase"],
+        leadingUnderscore: "allow",
+      },
+    ],
+    "@typescript-eslint/no-dynamic-delete": "warn",
+    "@typescript-eslint/no-empty-function": "off",
+    "@typescript-eslint/no-empty-interface": "off",
+    "@typescript-eslint/no-explicit-any": "off",
+    "@typescript-eslint/no-floating-promises": "warn",
+    "@typescript-eslint/no-parameter-properties": "warn",
+    "@typescript-eslint/no-shadow": "warn",
+    "@typescript-eslint/no-unused-vars": ["warn", { argsIgnorePattern: "^_", varsIgnorePattern: "^_" }],
+    "@typescript-eslint/no-unnecessary-type-assertion": "warn",
+    "@typescript-eslint/no-use-before-define": "warn",
+    "@typescript-eslint/prefer-readonly": "warn",
+  },
+  overrides: [
+    {
+      files: "**/*.js",
+      rules: {
+        "@typescript-eslint/no-var-requires": "off",
+        "@typescript-eslint/explicit-function-return-type": "off",
+        "@typescript-eslint/explicit-member-accessibility": "off",
+      },
+    },
+    {
+      files: "**/*.spec.ts",
+      rules: {
+        "@typescript-eslint/no-non-null-assertion": "off",
+      },
+    },
+  ],
+};