@pro-vision/eslint-config-pv - npm Package Compare versions

Comparing version 3.2.0 to 4.0.0

CHANGELOG.md

 # Changelog
 
+## 4.0.0
+
+- Updated: updated for @typescript-eslint v6
+
 ## 3.2.0
@@ -4,0 +8,0 @@

package.json

 {
   "name": "@pro-vision/eslint-config-pv",
-  "version": "3.2.0",
+  "version": "4.0.0",
   "description": "pro!vision ESLint configuration. Can be adapted and adjusted in each project.",
@@ -45,3 +45,3 @@ "author": {
   },
-  "main": "es6.js",
+  "main": "index.js",
   "keywords": [
@@ -56,5 +56,5 @@ "eslint",
     "@types/node": "^16.0.0",
-    "@typescript-eslint/eslint-plugin": "5.60.1",
-    "@typescript-eslint/parser": "5.60.1",
-    "eslint": "8.44.0",
+    "@typescript-eslint/eslint-plugin": "6.1.0",
+    "@typescript-eslint/parser": "6.1.0",
+    "eslint": "8.45.0",
     "eslint-config-prettier": "8.8.0",
@@ -61,0 +61,0 @@ "eslint-plugin-import": "2.27.5",

README.md

@@ -7,2 +7,7 @@ # @pro-vision/eslint-config-pv
 
+Rules mostly follow:
+
+- eslint:recommended
+- @typescript-eslint/recommended
+
 ## Versions
@@ -13,2 +18,3 @@
 ## Installation
+
 ```bash
@@ -20,3 +26,2 @@ npm install --save-dev @pro-vision/eslint-config-pv eslint-plugin-import
 
-
 ## Usage
@@ -26,3 +31,2 @@
 
-
 ### eslint-config-pv
@@ -62,3 +66,2 @@
 
-
 ```bash
@@ -79,3 +82,2 @@ npm install --save-dev eslint-config-prettier eslint-plugin-prettier prettier
 
-
 ### eslint-config-pv/typescript
@@ -115,3 +117,5 @@
 ## WebStorm Integration
+
 Ensure you are using `node >= 4.5` and you have installed `eslint` and `eslint-plugin-import` globally:
+
 ```bash
@@ -118,0 +122,0 @@ # node version should be at least 4.5

rules/ts-reset.js

@@ -44,3 +44,6 @@ // disable all rules known to conflict with @typescript-eslint plugin
     "space-before-blocks": "off",
+    "block-spacing": "off",
+    "key-spacing": "off",
+    "lines-around-comment": "off",
   },
 };

rules/ts.js

@@ -33,7 +33,6 @@ module.exports = {
     "@typescript-eslint/no-array-constructor": ["error"],
-    "@typescript-eslint/no-base-to-string": "off",
+    "@typescript-eslint/no-base-to-string": "error",
     "@typescript-eslint/no-confusing-non-null-assertion": "off",
     "@typescript-eslint/no-confusing-void-expression": "off",
     "@typescript-eslint/no-dupe-class-members": ["error"],
-    "@typescript-eslint/no-duplicate-imports": "off",
     "@typescript-eslint/no-dynamic-delete": "off",
@@ -43,3 +42,3 @@ "@typescript-eslint/no-empty-function": ["error"],
     "@typescript-eslint/no-explicit-any": "warn",
-    "@typescript-eslint/no-extra-non-null-assertion": "off",
+    "@typescript-eslint/no-extra-non-null-assertion": "error",
     "@typescript-eslint/no-extra-parens": ["off"],
@@ -50,3 +49,2 @@ "@typescript-eslint/no-extra-semi": ["error"],
     "@typescript-eslint/no-for-in-array": "error",
-    "@typescript-eslint/no-implicit-any-catch": "off",
     "@typescript-eslint/no-implied-eval": ["error"],
@@ -57,3 +55,3 @@ "@typescript-eslint/no-inferrable-types": "error",
     "@typescript-eslint/no-loop-func": "off",
-    "@typescript-eslint/no-loss-of-precision": "off",
+    "@typescript-eslint/no-loss-of-precision": "error",
     "@typescript-eslint/no-magic-numbers": ["off"],
@@ -63,6 +61,5 @@ "@typescript-eslint/no-misused-new": "error",
     "@typescript-eslint/no-namespace": "error",
-    "@typescript-eslint/no-non-null-asserted-optional-chain": "off",
-    // no additional linting when the developer has has decided to use non null assertion
+    "@typescript-eslint/no-non-null-asserted-optional-chain": "error",
+    // no additional linting when the developer has decided to use non null assertion
     "@typescript-eslint/no-non-null-assertion": "off",
-    "@typescript-eslint/no-parameter-properties": "off",
     "@typescript-eslint/no-redeclare": "off",
@@ -118,3 +115,2 @@ "@typescript-eslint/no-require-imports": "off",
     "@typescript-eslint/semi": ["error", "always"],
-    "@typescript-eslint/sort-type-union-intersection-members": "off",
     "@typescript-eslint/space-before-function-paren": ["off"],
@@ -142,12 +138,21 @@ "@typescript-eslint/space-infix-ops": "off",
     "@typescript-eslint/space-before-blocks": "off",
-    "@typescript-eslint/block-spacing": "off",
-    "@typescript-eslint/key-spacing": "off",
-    "@typescript-eslint/lines-around-comment": "off",
-    "@typescript-eslint/no-duplicate-type-constituents": "off",
+    // similar to the rule from 4-0.js (will be removed in combination with eslint-config-pv/prettier)
+    "@typescript-eslint/block-spacing": ["error", "always"],
+    // similar to the rule from 4-0.js (will be removed in combination with eslint-config-pv/prettier)
+    "@typescript-eslint/key-spacing": ["warn", { "beforeColon": false, "afterColon": true }],
+    // similar to the rule from 4-0.js (will be removed in combination with eslint-config-pv/prettier)
+    "@typescript-eslint/lines-around-comment": ["warn", {
+      "beforeBlockComment": true,
+      "beforeLineComment": false,
+      "allowClassStart": true,
+      "allowObjectStart": true,
+      "allowArrayStart": true,
+    }],
+    "@typescript-eslint/no-duplicate-type-constituents": "error",
     "@typescript-eslint/no-import-type-side-effects": "off",
     "@typescript-eslint/no-mixed-enums": "off",
-    "@typescript-eslint/no-unsafe-declaration-merging": "off",
-    "@typescript-eslint/no-unsafe-enum-comparison": "off",
+    "@typescript-eslint/no-unsafe-declaration-merging": "error",
+    "@typescript-eslint/no-unsafe-enum-comparison": "error",
     "@typescript-eslint/sort-type-constituents": "off",
   }
 };

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

64251

increased by0.85%

Lines of code

1005

increased by0.8%

Number of lines in readme file

130

increased by3.17%

No dependency changes