@metamask/eslint-config-nodejs - npm Package Compare versions

Comparing version 11.1.0 to 12.0.0

CHANGELOG.md

@@ -9,2 +9,7 @@ # Changelog
 
+## [12.0.0]
+### Changed
+- **BREAKING:** Update peer dependency `@metamask/eslint-config` to v12
+- **BREAKING:** Replace `eslint-plugin-node` with `eslint-plugin-n` ([#297](https://github.com/MetaMask/eslint-config/pull/297))
+
 ## [11.1.0]
@@ -70,3 +75,4 @@ ### Changed
 
-[Unreleased]: https://github.com/MetaMask/eslint-config/compare/v11.1.0...HEAD
+[Unreleased]: https://github.com/MetaMask/eslint-config/compare/v12.0.0...HEAD
+[12.0.0]: https://github.com/MetaMask/eslint-config/compare/v11.1.0...v12.0.0
 [11.1.0]: https://github.com/MetaMask/eslint-config/compare/v11.0.1...v11.1.0
@@ -73,0 +79,0 @@ [11.0.1]: https://github.com/MetaMask/eslint-config/compare/v11.0.0...v11.0.1

package.json

 {
   "name": "@metamask/eslint-config-nodejs",
-  "version": "11.1.0",
+  "version": "12.0.0",
   "description": "Shareable MetaMask ESLint config for Node.js.",
@@ -26,3 +26,3 @@ "homepage": "https://github.com/MetaMask/eslint-config#readme",
     "@metamask/auto-changelog": "^3.0.0",
-    "@metamask/eslint-config": "^11.0.0",
+    "@metamask/eslint-config": "^12.0.0",
     "eslint": "^8.27.0",
@@ -32,3 +32,3 @@ "eslint-config-prettier": "^8.5.0",
     "eslint-plugin-jsdoc": "^39.6.2",
-    "eslint-plugin-node": "^11.1.0",
+    "eslint-plugin-n": "^15.7.0",
     "eslint-plugin-prettier": "^4.2.1",
@@ -38,5 +38,5 @@ "prettier": "^2.7.1"
   "peerDependencies": {
-    "@metamask/eslint-config": "^11.0.0",
+    "@metamask/eslint-config": "^12.0.0",
     "eslint": "^8.27.0",
-    "eslint-plugin-node": "^11.1.0"
+    "eslint-plugin-n": "^15.7.0"
   },
@@ -43,0 +43,0 @@ "engines": {

README.md

@@ -9,4 +9,4 @@ # `@metamask/eslint-config-nodejs`
 yarn add --dev \
-    @metamask/eslint-config@^11.0.1 \
-    @metamask/eslint-config-nodejs@^11.0.1 \
+    @metamask/eslint-config@^12.0.0 \
+    @metamask/eslint-config-nodejs@^12.0.0 \
     eslint@^8.27.0 \
@@ -16,4 +16,5 @@ eslint-config-prettier@^8.5.0 \
     eslint-plugin-jsdoc@^39.6.2 \
-    eslint-plugin-node@^11.1.0 \
+    eslint-plugin-n@^15.7.0 \
     eslint-plugin-prettier@^4.2.1 \
+    eslint-plugin-promise@^6.1.1 \
     prettier@^2.7.1
@@ -20,0 +21,0 @@ ```

src/index.js

 const environmentRules = require('./environment.json');
 
 module.exports = {
-  plugins: ['node'],
+  plugins: ['n'],
 
@@ -21,3 +21,3 @@ env: {
 
-  extends: ['plugin:node/recommended'],
+  extends: ['plugin:n/recommended'],
 
@@ -28,34 +28,29 @@ rules: {
     // Possible Errors
-    'node/handle-callback-err': ['error', '^(err|error)$'],
-    'node/no-callback-literal': 'error',
-    'node/no-missing-import': 'off', // Duplicates `import/no-unresolved`
-    'node/no-missing-require': 'off', // Duplicates `import/no-unresolved`
-    'node/no-new-require': 'error',
-    'node/no-path-concat': 'error',
-    'node/no-process-exit': 'error',
-    'node/no-unsupported-features/es-syntax': 'off',
+    'n/handle-callback-err': ['error', '^(err|error)$'],
+    'n/no-callback-literal': 'error',
+    'n/no-missing-import': 'off', // Duplicates `import/no-unresolved`
+    'n/no-missing-require': 'off', // Duplicates `import/no-unresolved`
+    'n/no-new-require': 'error',
+    'n/no-path-concat': 'error',
+    'n/no-unsupported-features/es-syntax': 'off',
 
     // Stylistic rules
-    'node/callback-return': 'error',
-    'node/exports-style': 'error',
-    'node/global-require': 'error',
-    'node/no-mixed-requires': 'error',
-    'node/no-process-env': 'error',
-    'node/no-restricted-import': 'error',
-    'node/no-restricted-require': 'error',
-    'node/no-sync': 'error',
-    'node/prefer-global/buffer': 'error',
-    'node/prefer-global/console': 'error',
-    'node/prefer-global/process': 'error',
-    'node/prefer-global/text-decoder': 'error',
-    'node/prefer-global/text-encoder': 'error',
-    'node/prefer-global/url-search-params': 'error',
-    'node/prefer-global/url': 'error',
-    'node/prefer-promises/dns': 'error',
-    'node/prefer-promises/fs': 'error',
+    'n/callback-return': 'error',
+    'n/exports-style': 'error',
+    'n/global-require': 'error',
+    'n/no-mixed-requires': 'error',
+    'n/no-process-env': 'error',
+    'n/no-restricted-import': 'error',
+    'n/no-restricted-require': 'error',
+    'n/no-sync': 'error',
+    'n/prefer-global/buffer': 'error',
+    'n/prefer-global/console': 'error',
+    'n/prefer-global/process': 'error',
+    'n/prefer-global/text-decoder': 'error',
+    'n/prefer-global/text-encoder': 'error',
+    'n/prefer-global/url-search-params': 'error',
+    'n/prefer-global/url': 'error',
+    'n/prefer-promises/dns': 'error',
+    'n/prefer-promises/fs': 'error',
 
-    // Deprecated eslint core rule, erroneously enabled by recommended Node rules
-    // https://eslint.org/docs/rules/no-process-exit
-    'no-process-exit': 'off',
-
     // Enabled in the base config, but this should be allowed in Node.js
@@ -62,0 +57,0 @@ // projects.

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

94599

increased by0.05%

Number of lines in readme file

37

increased by2.78%

Worsened metrics

Lines of code

2851

decreased by-0.14%

No dependency changes