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

Comparing version 12.1.0 to 11.1.0

CHANGELOG.md

@@ -9,11 +9,2 @@ # Changelog
 
-## [12.1.0]
-### Changed
-- Add support for typescript 5.0.x, 5.1.x ([#288](https://github.com/MetaMask/eslint-config/pull/288))
-
-## [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]
@@ -79,5 +70,3 @@ ### Changed
 
-[Unreleased]: https://github.com/MetaMask/eslint-config/compare/v12.1.0...HEAD
-[12.1.0]: https://github.com/MetaMask/eslint-config/compare/v12.0.0...v12.1.0
-[12.0.0]: https://github.com/MetaMask/eslint-config/compare/v11.1.0...v12.0.0
+[Unreleased]: https://github.com/MetaMask/eslint-config/compare/v11.1.0...HEAD
 [11.1.0]: https://github.com/MetaMask/eslint-config/compare/v11.0.1...v11.1.0
@@ -84,0 +73,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": "12.1.0",
+  "version": "11.1.0",
   "description": "Shareable MetaMask ESLint config for Node.js.",
@@ -26,8 +26,8 @@ "homepage": "https://github.com/MetaMask/eslint-config#readme",
     "@metamask/auto-changelog": "^3.0.0",
-    "@metamask/eslint-config": "^12.0.0",
+    "@metamask/eslint-config": "^11.0.0",
     "eslint": "^8.27.0",
     "eslint-config-prettier": "^8.5.0",
-    "eslint-plugin-import": "^2.27.5",
-    "eslint-plugin-jsdoc": "^41.1.2",
-    "eslint-plugin-n": "^15.7.0",
+    "eslint-plugin-import": "^2.26.0",
+    "eslint-plugin-jsdoc": "^39.6.2",
+    "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-prettier": "^4.2.1",
@@ -37,5 +37,5 @@ "prettier": "^2.7.1"
   "peerDependencies": {
-    "@metamask/eslint-config": "^12.0.0",
+    "@metamask/eslint-config": "^11.0.0",
     "eslint": "^8.27.0",
-    "eslint-plugin-n": "^15.7.0"
+    "eslint-plugin-node": "^11.1.0"
   },
@@ -42,0 +42,0 @@ "engines": {

README.md

@@ -9,11 +9,10 @@ # `@metamask/eslint-config-nodejs`
 yarn add --dev \
-    @metamask/eslint-config@^12.0.0 \
-    @metamask/eslint-config-nodejs@^12.0.0 \
+    @metamask/eslint-config@^11.0.1 \
+    @metamask/eslint-config-nodejs@^11.0.1 \
     eslint@^8.27.0 \
     eslint-config-prettier@^8.5.0 \
-    eslint-plugin-import@^2.27.5 \
-    eslint-plugin-jsdoc@^41.1.2 \
-    eslint-plugin-n@^15.7.0 \
+    eslint-plugin-import@^2.26.0 \
+    eslint-plugin-jsdoc@^39.6.2 \
+    eslint-plugin-node@^11.1.0 \
     eslint-plugin-prettier@^4.2.1 \
-    eslint-plugin-promise@^6.1.1 \
     prettier@^2.7.1
@@ -20,0 +19,0 @@ ```

src/index.js

 const environmentRules = require('./environment.json');
 
 module.exports = {
-  plugins: ['n'],
+  plugins: ['node'],
 
@@ -21,3 +21,3 @@ env: {
 
-  extends: ['plugin:n/recommended'],
+  extends: ['plugin:node/recommended'],
 
@@ -28,29 +28,34 @@ rules: {
     // Possible Errors
-    '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',
+    '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',
 
     // Stylistic rules
-    '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',
+    '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',
 
+    // 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
@@ -57,0 +62,0 @@ // projects.

No alert changes

Improved metrics

Lines of code

2855

0.14%

Worsened metrics

Total package byte prevSize

94550

-0.27%

Number of lines in readme file

36

-2.7%

No dependency changes