Merge pull request #213 from StefanoMagrassi/main

Author: gabro

.eslintrc.json

@@ -3,8 +3,11 @@
   "plugins": ["fp-ts"],
   "extends": "plugin:fp-ts/all",
   "rules": {
-    "fp-ts/no-module-imports": ["error", {
-      "allowTypes": true
-    }]
+    "fp-ts/no-module-imports": [
+      "error",
+      {
+        "allowTypes": true
+      }
+    ]
   }
 }

package.json

@@ -15,6 +15,7 @@
     "url": "https://github.com/buildo/eslint-plugin-fp-ts"
   },
   "scripts": {
+    "prepare": "npm run build",
     "preversion": "yarn lint && yarn test && yarn build",
     "build": "rimraf lib && yarn tsc",
     "pretest": "yarn --cwd tests/fixtures/fp-ts-project install",
@@ -25,7 +26,7 @@
     "lib"
   ],
   "dependencies": {
-    "@typescript-eslint/experimental-utils": "^4.11.1",
+    "@typescript-eslint/experimental-utils": "^5.0.0",
     "estraverse": "^5.2.0",
     "fp-ts": "^2.9.3",
     "recast": "^0.20.4"
@@ -36,10 +37,10 @@
     "@types/estraverse": "^5.1.0",
     "@types/node": "^14.14.16",
     "@types/requireindex": "^1.2.0",
-    "@typescript-eslint/parser": "^4.11.1",
+    "@typescript-eslint/parser": "^5.0.0",
     "common-tags": "^1.8.0",
-    "eslint": "^7.17.0",
-    "eslint-plugin-fp-ts": "^0.1.8",
+    "eslint": "^8.0.0",
+    "eslint-plugin-fp-ts": "link:./lib",
     "jest": "^26.6.3",
     "rimraf": "^3.0.2",
     "ts-jest": "^26.4.4",

src/rules/no-discarded-pure-expression.ts

@@ -12,9 +12,9 @@ export default createRule({
   name: "no-discarded-pure-expression",
   meta: {
     type: "problem",
+    hasSuggestions: true,
     schema: [],
     docs: {
-      category: "Possible Errors",
       description:
         "Detects pure expressions that do nothing because they're in statement position",
       recommended: "error",

src/rules/no-lib-imports.ts

@@ -7,7 +7,6 @@ export default createRule({
     type: "problem",
     fixable: "code",
     docs: {
-      category: "Possible Errors",
       description: "Disallow imports from 'fp-ts/lib'",
       recommended: "error",
     },

src/rules/no-module-imports.ts

@@ -22,7 +22,6 @@ export default createRule<Options, MessageIds>({
     type: "problem",
     fixable: "code",
     docs: {
-      category: "Best Practices",
       description:
         "Disallow imports from fp-ts modules, such as `fp-ts/Option`",
       recommended: "error",

src/rules/no-pipeable.ts

@@ -8,7 +8,6 @@ export default createRule({
     fixable: "code",
     schema: [],
     docs: {
-      category: "Possible Errors",
       description: "Disallow imports from the 'pipeable' module",
       recommended: "error",
     },

src/rules/no-redundant-flow.ts

@@ -5,9 +5,9 @@ export default createRule({
   meta: {
     type: "suggestion",
     fixable: "code",
+    hasSuggestions: true,
     schema: [],
     docs: {
-      category: "Best Practices",
       description: "Remove redundant uses of flow",
       recommended: "warn",
     },

src/rules/prefer-bimap.ts

@@ -18,9 +18,9 @@ export default createRule({
   meta: {
     type: "suggestion",
     fixable: "code",
+    hasSuggestions: true,
     schema: [],
     docs: {
-      category: "Best Practices",
       description: "Replace map + mapLeft with bimap",
       recommended: "warn",
     },

src/rules/prefer-chain.ts

@@ -13,9 +13,9 @@ export default createRule({
   meta: {
     type: "suggestion",
     fixable: "code",
+    hasSuggestions: true,
     schema: [],
     docs: {
-      category: "Best Practices",
       description: "Replace map + flatten with chain",
       recommended: "warn",
     },

src/rules/prefer-traverse.ts

@@ -17,9 +17,9 @@ export default createRule({
   meta: {
     type: "suggestion",
     fixable: "code",
+    hasSuggestions: true,
     schema: [],
     docs: {
-      category: "Best Practices",
       description: "Replace map + sequence with traverse",
       recommended: "warn",
     },