build(deps): update latest

BREAKING CHANGE: peerDependencies: `eslint-plugin-unicorn@^46.0.0`

.eslintrc.json

@@ -34,6 +34,7 @@
     "no-duplicate-imports": "error",
     "no-empty-character-class": "error",
     "no-empty-pattern": "error",
+    "no-empty-static-block": "error",
     "no-ex-assign": "error",
     "no-fallthrough": "error",
     "no-func-assign": "error",
@@ -42,6 +43,7 @@
     "no-irregular-whitespace": "error",
     "no-loss-of-precision": "error",
     "no-misleading-character-class": "error",
+    "no-new-native-nonconstructor": "error",
     "no-new-symbol": "error",
     "no-obj-calls": "error",
     "no-promise-executor-return": "error",
@@ -59,7 +61,7 @@
     "no-unsafe-negation": "error",
     "no-unsafe-optional-chaining": "error",
     "no-unused-private-class-members": "error",
-    "no-unused-vars": "error",
+    "no-unused-vars": ["error", { "varsIgnorePattern": "^_" }],
     "no-use-before-define": [
       "error",
       {
@@ -145,6 +147,11 @@
     "quote-props": ["error", "as-needed"],
     "radix": "error",
     "yoda": ["error", "never"],
+    "curly": ["error", "all"],
+    "func-style": ["error", "expression"],
+    "prefer-arrow-callback": "error",
+    "arrow-parens": ["error", "always"],
+    "arrow-body-style": ["error", "always"],
 
     "import/no-absolute-path": "error",
     "import/no-webpack-loader-syntax": "error",
@@ -161,6 +168,7 @@
       }
     ],
     "import/no-named-default": "error",
+    "import/no-empty-named-blocks": "error",
 
     "promise/param-names": "error",
     "promise/no-new-statics": "error",
@@ -176,7 +184,8 @@
     "unicorn/no-this-assignment": "error",
     "unicorn/no-zero-fractions": "error",
     "unicorn/prefer-node-protocol": "error",
-    "unicorn/throw-new-error": "error"
+    "unicorn/throw-new-error": "error",
+    "unicorn/no-typeof-undefined": "error"
   },
   "overrides": [
     {
@@ -192,7 +201,10 @@
         "no-loss-of-precision": "off",
         "@typescript-eslint/no-loss-of-precision": "error",
         "no-unused-vars": "off",
-        "@typescript-eslint/no-unused-vars": "error",
+        "@typescript-eslint/no-unused-vars": [
+          "error",
+          { "varsIgnorePattern": "^_" }
+        ],
         "no-use-before-define": "off",
         "@typescript-eslint/no-use-before-define": [
           "error",
@@ -236,17 +248,12 @@
           "error",
           { "default": "array-simple" }
         ],
-        "@typescript-eslint/consistent-type-assertions": [
-          "error",
-          {
-            "assertionStyle": "as",
-            "objectLiteralTypeAssertions": "never"
-          }
-        ],
+        "@typescript-eslint/ban-types": "error",
         "@typescript-eslint/consistent-type-definitions": [
           "error",
           "interface"
         ],
+        "@typescript-eslint/consistent-type-imports": "error",
         "@typescript-eslint/explicit-function-return-type": [
           "error",
           {
@@ -275,10 +282,6 @@
         ],
         "@typescript-eslint/no-base-to-string": "error",
         "@typescript-eslint/no-dynamic-delete": "error",
-        "@typescript-eslint/no-empty-interface": [
-          "error",
-          { "allowSingleExtends": true }
-        ],
         "@typescript-eslint/no-extra-non-null-assertion": "error",
         "@typescript-eslint/no-extraneous-class": [
           "error",
@@ -306,7 +309,6 @@
             "ignoreMixedLogicalExpressions": false
           }
         ],
-        "@typescript-eslint/prefer-optional-chain": "error",
         "@typescript-eslint/prefer-readonly": "error",
         "@typescript-eslint/prefer-reduce-type-parameter": "error",
         "@typescript-eslint/prefer-ts-expect-error": "error",

.github/workflows/lint.yml

@@ -10,10 +10,10 @@ jobs:
   lint:
     runs-on: 'ubuntu-latest'
     steps:
-      - uses: 'actions/checkout@v3.0.0'
+      - uses: 'actions/checkout@v3.5.0'
 
       - name: 'Use Node.js'
-        uses: 'actions/setup-node@v3.1.0'
+        uses: 'actions/setup-node@v3.6.0'
         with:
           node-version: 'lts/*'
           cache: 'npm'
@@ -24,5 +24,5 @@ jobs:
       - run: 'npm run lint:commit -- --to "${{ github.sha }}"'
       - run: 'npm run lint:editorconfig'
       - run: 'npm run lint:markdown'
-      - run: 'npm run lint:javascript'
+      - run: 'npm run lint:eslint'
       - run: 'npm run lint:prettier'

.github/workflows/release.yml

@@ -8,13 +8,13 @@ jobs:
   release:
     runs-on: 'ubuntu-latest'
     steps:
-      - uses: 'actions/checkout@v3.0.0'
+      - uses: 'actions/checkout@v3.5.0'
         with:
           fetch-depth: 0
           persist-credentials: false
 
       - name: 'Use Node.js'
-        uses: 'actions/setup-node@v3.1.0'
+        uses: 'actions/setup-node@v3.6.0'
         with:
           node-version: 'lts/*'
           cache: 'npm'

.github/workflows/test.yml

@@ -10,10 +10,10 @@ jobs:
   test:
     runs-on: 'ubuntu-latest'
     steps:
-      - uses: 'actions/checkout@v3.0.0'
+      - uses: 'actions/checkout@v3.5.0'
 
       - name: 'Use Node.js'
-        uses: 'actions/setup-node@v3.1.0'
+        uses: 'actions/setup-node@v3.6.0'
         with:
           node-version: 'lts/*'
           cache: 'npm'

.lintstagedrc.json

@@ -2,5 +2,5 @@
   "*": ["editorconfig-checker"],
   "*.{js,jsx,ts,tsx}": ["prettier --write", "eslint --fix"],
   "*.{json,jsonc,yml,yaml}": ["prettier --write"],
-  "*.{md,mdx}": ["prettier --write", "markdownlint --dot --fix"]
+  "*.{md,mdx}": ["prettier --write", "markdownlint-cli2 --fix"]
 }

.markdownlint-cli2.jsonc

@@ -0,0 +1,5 @@
+{
+  "globs": ["**/*.{md,mdx}"],
+  "ignores": ["**/node_modules"],
+  "customRules": ["markdownlint-rule-relative-links"]
+}

.markdownlint.json

@@ -1,6 +1,7 @@
 {
   "default": true,
-  "MD013": false,
+  "relative-links": true,
+  "extends": "markdownlint/style/prettier",
   "MD033": false,
   "MD041": false
 }

README.md

@@ -23,7 +23,7 @@
 
 **eslint-config-conventions** is a [ESLint](https://eslint.org) configuration to enforce strict conventions and good code quality, it is highly inspired from [eslint-config-standard-with-typescript](https://github.com/standard/eslint-config-standard-with-typescript) but it is **stricter** and with **no formatting rules**, **only code-quality rules**. It supports both **JavaScript** and **TypeScript**.
 
-This configuration is mostly **for catching bugs** and **code-quality** so it is recommanded to use it with [Prettier](https://prettier.io/) for a consistent code style, it works with any `.prettierrc.json` configuration.
+This configuration is mostly **for catching bugs** and **code-quality** so it is recommended to use it with [Prettier](https://prettier.io/) for a consistent code style, it works with any `.prettierrc.json` configuration.
 
 More information about **formatting rules** vs **code-quality rules** can be found on [Prettier vs. Linters](https://prettier.io/docs/en/comparison.html).
 
@@ -40,10 +40,10 @@ Here is an example, but use it only for reference, because your decisions regard
 
 ```sh
 npm install --save-dev \
-  eslint@^8.9.0 \
-  eslint-plugin-import@^2.26.0 \
-  eslint-plugin-promise@^6.0.0 \
-  eslint-plugin-unicorn@^43.0.0 \
+  eslint@^8.33.0 \
+  eslint-plugin-import@^2.27.5 \
+  eslint-plugin-promise@^6.1.1 \
+  eslint-plugin-unicorn@^46.0.0 \
   eslint-config-conventions@latest
 ```
 
@@ -62,9 +62,9 @@ If you want to use **TypeScript**, you also need to install:
 
 ```sh
 npm install --save-dev \
-  typescript@^4.6.0 \
-  @typescript-eslint/eslint-plugin@^5.18.0 \
-  @typescript-eslint/parser@^5.18.0
+  typescript@^5.0.3 \
+  @typescript-eslint/eslint-plugin@^5.57.0 \
+  @typescript-eslint/parser@^5.57.0
 ```
 
 Dependencies are:
@@ -109,7 +109,7 @@ echo {}> .prettierrc.json
 ```jsonc
 {
   "scripts": {
-    "lint:typescript": "eslint \"**/*.{js,jsx,ts,tsx}\" --ignore-path \".gitignore\"",
+    "lint:eslint": "eslint \"**/*.{js,jsx,ts,tsx}\" --ignore-path \".gitignore\"",
     "lint:prettier": "prettier \".\" --check --ignore-path \".gitignore\""
   }
 }
@@ -118,9 +118,9 @@ echo {}> .prettierrc.json
 ### Usage
 
 ```sh
-npm run lint:typescript
+npm run lint:eslint
 # or to apply automatic fixes to code
-npm run lint:typescript -- --fix
+npm run lint:eslint -- --fix
 
 # Validate code formatting in all supported languages by Prettier
 npm run lint:prettier

package.json

@@ -23,6 +23,10 @@
     "conventions"
   ],
   "main": "index.js",
+  "files": [
+    "index.js",
+    ".eslintrc.json"
+  ],
   "engines": {
     "node": ">=16.0.0",
     "npm": ">=8.0.0"
@@ -30,8 +34,8 @@
   "scripts": {
     "lint:commit": "commitlint",
     "lint:editorconfig": "editorconfig-checker",
-    "lint:markdown": "markdownlint \"**/*.md\" --dot --ignore-path \".gitignore\"",
-    "lint:javascript": "eslint \"**/*.{js,jsx,ts,tsx}\" -c \"eslintrc.json\"",
+    "lint:markdown": "markdownlint-cli2",
+    "lint:eslint": "eslint \"**/*.{js,jsx,ts,tsx}\" -c \"eslintrc.json\"",
     "lint:prettier": "prettier \".\" --check --ignore-path \".gitignore\"",
     "lint:staged": "lint-staged",
     "test": "tap",
@@ -41,30 +45,32 @@
     "postpublish": "pinst --enable"
   },
   "peerDependencies": {
-    "eslint": "^8.9.0",
-    "eslint-plugin-import": "^2.26.0",
-    "eslint-plugin-promise": "^6.0.0",
-    "eslint-plugin-unicorn": "^43.0.0"
+    "eslint": "^8.33.0",
+    "eslint-plugin-import": "^2.27.5",
+    "eslint-plugin-promise": "^6.1.1",
+    "eslint-plugin-unicorn": "^46.0.0"
   },
   "devDependencies": {
-    "@commitlint/cli": "17.0.3",
-    "@commitlint/config-conventional": "17.0.3",
-    "@types/eslint": "8.4.5",
-    "@types/tap": "15.0.7",
-    "@typescript-eslint/eslint-plugin": "5.30.6",
-    "@typescript-eslint/parser": "5.30.6",
-    "editorconfig-checker": "4.0.2",
-    "eslint": "8.19.0",
-    "eslint-plugin-import": "2.26.0",
-    "eslint-plugin-promise": "6.0.0",
-    "eslint-plugin-unicorn": "43.0.1",
-    "husky": "8.0.1",
-    "lint-staged": "13.0.3",
-    "markdownlint-cli": "0.31.1",
+    "@commitlint/cli": "17.5.1",
+    "@commitlint/config-conventional": "17.4.4",
+    "@tsconfig/strictest": "2.0.0",
+    "@types/eslint": "8.37.0",
+    "@types/tap": "15.0.8",
+    "@typescript-eslint/eslint-plugin": "5.57.0",
+    "@typescript-eslint/parser": "5.57.0",
+    "editorconfig-checker": "5.0.1",
+    "eslint": "8.37.0",
+    "eslint-plugin-import": "2.27.5",
+    "eslint-plugin-promise": "6.1.1",
+    "eslint-plugin-unicorn": "46.0.0",
+    "husky": "8.0.3",
+    "lint-staged": "13.2.0",
+    "markdownlint-cli2": "0.6.0",
+    "markdownlint-rule-relative-links": "1.1.1",
     "pinst": "3.0.0",
-    "prettier": "2.7.1",
-    "semantic-release": "19.0.3",
-    "tap": "16.3.0",
-    "typescript": "4.7.4"
+    "prettier": "2.8.7",
+    "semantic-release": "21.0.1",
+    "tap": "16.3.4",
+    "typescript": "5.0.3"
   }
 }

test/basic.js

@@ -2,13 +2,13 @@ const tap = require('tap')
 
 const config = require('../index.js')
 
+const isObject = (object) => {
+  return typeof object === 'object' && object !== null
+}
+
 tap.test('test basic properties of config', async (t) => {
   t.ok(isObject(config.parserOptions))
   t.ok(isObject(config.env))
   t.ok(isObject(config.rules))
   t.ok(isObject(config.overrides))
 })
-
-function isObject(object) {
-  return typeof object === 'object' && object !== null
-}

test/fixtures/top-level-await.mjs

@@ -1,3 +1,3 @@
 const foo = await 1
-const bar = function () {}
+const bar = () => {}
 await bar(foo)

tsconfig.json

@@ -1,12 +1,10 @@
 {
+  "extends": "@tsconfig/strictest/tsconfig.json",
   "compilerOptions": {
     "target": "ESNext",
     "module": "commonjs",
     "lib": ["ESNext"],
     "moduleResolution": "node",
-    "rootDir": "./",
-    "strict": true,
-    "skipLibCheck": true,
-    "esModuleInterop": true
+    "rootDir": "./"
   }
 }