build(deps): update latest

BREAKING CHANGE: peerDependencies: `eslint-plugin-import@^2.26.0`, `eslint-plugin-unicorn@^42.0.0`

.eslintrc.json

@@ -1,19 +1,17 @@
 {
   "root": true,
-  "plugins": ["@typescript-eslint", "import", "promise", "unicorn"],
+  "plugins": ["import", "promise", "unicorn"],
   "env": {
     "browser": true,
     "node": true,
     "es2022": true
   },
-  "parser": "@typescript-eslint/parser",
   "parserOptions": {
     "ecmaVersion": 2022,
     "sourceType": "module",
     "ecmaFeatures": {
       "jsx": true
-    },
-    "project": "./tsconfig.json"
+    }
   },
   "rules": {
     "constructor-super": "error",
@@ -29,13 +27,11 @@
     "no-control-regex": "error",
     "no-debugger": "error",
     "no-dupe-args": "error",
-    "no-dupe-class-members": "off",
-    "@typescript-eslint/no-dupe-class-members": "error",
+    "no-dupe-class-members": "error",
     "no-dupe-else-if": "error",
     "no-dupe-keys": "error",
     "no-duplicate-case": "error",
-    "no-duplicate-imports": "off",
-    "@typescript-eslint/no-duplicate-imports": "error",
+    "no-duplicate-imports": "error",
     "no-empty-character-class": "error",
     "no-empty-pattern": "error",
     "no-ex-assign": "error",
@@ -44,8 +40,7 @@
     "no-import-assign": "error",
     "no-invalid-regexp": "error",
     "no-irregular-whitespace": "error",
-    "no-loss-of-precision": "off",
-    "@typescript-eslint/no-loss-of-precision": "error",
+    "no-loss-of-precision": "error",
     "no-misleading-character-class": "error",
     "no-new-symbol": "error",
     "no-obj-calls": "error",
@@ -64,21 +59,16 @@
     "no-unsafe-negation": "error",
     "no-unsafe-optional-chaining": "error",
     "no-unused-private-class-members": "error",
-    "no-unused-vars": "off",
-    "@typescript-eslint/no-unused-vars": "error",
-    "no-use-before-define": "off",
-    "@typescript-eslint/no-use-before-define": [
+    "no-unused-vars": "error",
+    "no-use-before-define": [
       "error",
       {
         "functions": false,
         "classes": false,
-        "enums": false,
-        "variables": false,
-        "typedefs": false
+        "variables": false
       }
     ],
     "no-useless-backreference": "error",
-    "require-atomic-updates": "error",
     "use-isnan": [
       "error",
       {
@@ -89,19 +79,16 @@
     "valid-typeof": ["error", { "requireStringLiterals": true }],
 
     "consistent-this": "error",
-    "default-param-last": "off",
-    "@typescript-eslint/default-param-last": "error",
+    "default-param-last": "error",
     "default-case-last": "error",
-    "dot-notation": "off",
-    "@typescript-eslint/dot-notation": ["error", { "allowKeywords": true }],
+    "dot-notation": ["error", { "allowKeywords": true }],
     "eqeqeq": ["error", "always", { "null": "ignore" }],
     "grouped-accessor-pairs": "error",
     "new-cap": [
       "error",
       { "newIsCap": true, "capIsNew": false, "properties": true }
     ],
-    "no-array-constructor": "off",
-    "@typescript-eslint/no-array-constructor": "error",
+    "no-array-constructor": "error",
     "no-caller": "error",
     "no-confusing-arrow": "error",
     "no-delete-var": "error",
@@ -109,8 +96,7 @@
     "no-eval": "error",
     "no-floating-decimal": "error",
     "no-global-assign": "error",
-    "no-implied-eval": "off",
-    "@typescript-eslint/no-implied-eval": "error",
+    "no-implied-eval": "error",
     "no-mixed-operators": [
       "error",
       {
@@ -128,15 +114,12 @@
     "no-octal": "error",
     "no-octal-escape": "error",
     "no-proto": "error",
-    "no-redeclare": "off",
-    "@typescript-eslint/no-redeclare": ["error", { "builtinGlobals": false }],
+    "no-redeclare": ["error", { "builtinGlobals": false }],
     "no-regex-spaces": "error",
     "no-shadow-restricted-names": "error",
-    "no-throw-literal": "off",
-    "@typescript-eslint/no-throw-literal": "error",
+    "no-throw-literal": "error",
     "no-undef-init": "error",
-    "no-unused-expressions": "off",
-    "@typescript-eslint/no-unused-expressions": [
+    "no-unused-expressions": [
       "error",
       {
         "allowShortCircuit": true,
@@ -146,8 +129,7 @@
     ],
     "no-useless-call": "error",
     "no-useless-computed-key": "error",
-    "no-useless-constructor": "off",
-    "@typescript-eslint/no-useless-constructor": "error",
+    "no-useless-constructor": "error",
     "no-useless-escape": "error",
     "no-useless-rename": "error",
     "no-useless-return": "error",
@@ -162,9 +144,6 @@
     "prefer-regex-literals": ["error", { "disallowRedundantWrapping": true }],
     "quote-props": ["error", "as-needed"],
     "radix": "error",
-    "require-await": "off",
-    "@typescript-eslint/require-await": "error",
-    "spaced-comment": ["error", "always"],
     "yoda": ["error", "never"],
 
     "import/no-absolute-path": "error",
@@ -202,7 +181,56 @@
   "overrides": [
     {
       "files": ["*.ts", "*.tsx"],
+      "parser": "@typescript-eslint/parser",
+      "plugins": ["@typescript-eslint"],
       "rules": {
+        "no-undef": "off",
+        "no-dupe-class-members": "off",
+        "@typescript-eslint/no-dupe-class-members": "error",
+        "no-duplicate-imports": "off",
+        "@typescript-eslint/no-duplicate-imports": "error",
+        "no-loss-of-precision": "off",
+        "@typescript-eslint/no-loss-of-precision": "error",
+        "no-unused-vars": "off",
+        "@typescript-eslint/no-unused-vars": "error",
+        "no-use-before-define": "off",
+        "@typescript-eslint/no-use-before-define": [
+          "error",
+          {
+            "functions": false,
+            "classes": false,
+            "enums": false,
+            "variables": false,
+            "typedefs": false
+          }
+        ],
+        "default-param-last": "off",
+        "@typescript-eslint/default-param-last": "error",
+        "dot-notation": "off",
+        "@typescript-eslint/dot-notation": ["error", { "allowKeywords": true }],
+        "no-array-constructor": "off",
+        "@typescript-eslint/no-array-constructor": "error",
+        "no-implied-eval": "off",
+        "@typescript-eslint/no-implied-eval": "error",
+        "no-redeclare": "off",
+        "@typescript-eslint/no-redeclare": [
+          "error",
+          { "builtinGlobals": false }
+        ],
+        "no-throw-literal": "off",
+        "@typescript-eslint/no-throw-literal": "error",
+        "no-unused-expressions": "off",
+        "@typescript-eslint/no-unused-expressions": [
+          "error",
+          {
+            "allowShortCircuit": true,
+            "allowTernary": true,
+            "allowTaggedTemplates": true
+          }
+        ],
+        "no-useless-constructor": "off",
+        "@typescript-eslint/no-useless-constructor": "error",
+
         "@typescript-eslint/adjacent-overload-signatures": "error",
         "@typescript-eslint/array-type": [
           "error",
@@ -260,7 +288,6 @@
         "@typescript-eslint/no-for-in-array": "error",
         "@typescript-eslint/no-invalid-void-type": "error",
         "@typescript-eslint/no-misused-new": "error",
-        "@typescript-eslint/no-misused-promises": "error",
         "@typescript-eslint/no-non-null-asserted-optional-chain": "error",
         "@typescript-eslint/no-non-null-assertion": "error",
         "@typescript-eslint/no-this-alias": [

.github/workflows/lint.yml

@@ -2,7 +2,7 @@ name: 'Lint'
 
 on:
   push:
-    branches: [master, develop]
+    branches: [develop]
   pull_request:
     branches: [master, develop]
 
@@ -10,10 +10,10 @@ jobs:
   lint:
     runs-on: 'ubuntu-latest'
     steps:
-      - uses: 'actions/checkout@v2.4.0'
+      - uses: 'actions/checkout@v3.0.0'
 
       - name: 'Use Node.js'
-        uses: 'actions/setup-node@v2.5.1'
+        uses: 'actions/setup-node@v3.1.0'
         with:
           node-version: 'lts/*'
           cache: 'npm'

.github/workflows/release.yml

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

.github/workflows/test.yml

@@ -2,7 +2,7 @@ name: 'Test'
 
 on:
   push:
-    branches: [master, develop]
+    branches: [develop]
   pull_request:
     branches: [master, develop]
 
@@ -10,10 +10,10 @@ jobs:
   test:
     runs-on: 'ubuntu-latest'
     steps:
-      - uses: 'actions/checkout@v2.4.0'
+      - uses: 'actions/checkout@v3.0.0'
 
       - name: 'Use Node.js'
-        uses: 'actions/setup-node@v2.5.1'
+        uses: 'actions/setup-node@v3.1.0'
         with:
           node-version: 'lts/*'
           cache: 'npm'

.gitignore

@@ -4,6 +4,7 @@ node_modules
 
 # testing
 coverage
+.nyc_output
 
 # debug
 npm-debug.log*

.taprc

@@ -0,0 +1,8 @@
+ts: false
+jsx: false
+flow: false
+check-coverage: true
+coverage: true
+
+test-ignore:
+  - 'test/fixtures'

README.md

@@ -21,7 +21,7 @@
 
 ## 📜 About
 
-**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**.
+**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.
 
@@ -41,11 +41,9 @@ 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.25.4 \
+  eslint-plugin-import@^2.26.0 \
   eslint-plugin-promise@^6.0.0 \
-  eslint-plugin-unicorn@^41.0.0 \
-  typescript@^4.4.0 \
-  @typescript-eslint/eslint-plugin@^5.12.0 \
+  eslint-plugin-unicorn@^42.0.0 \
   eslint-config-conventions@latest
 ```
 
@@ -58,9 +56,22 @@ This list of dependencies are:
   - [eslint-plugin-import](https://github.com/import-js/eslint-plugin-import)
   - [eslint-plugin-promise](https://github.com/xjamundx/eslint-plugin-promise)
   - [eslint-plugin-unicorn](https://github.com/sindresorhus/eslint-plugin-unicorn)
+- This package: `eslint-config-conventions`
+
+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
+```
+
+Dependencies are:
+
 - [TypeScript](https://github.com/Microsoft/TypeScript)
 - [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint); ESLint rules for TypeScript.
-- This package: `eslint-config-conventions`
+- [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint); ESLint parser for TypeScript.
 
 ### Configuration with [Prettier](https://prettier.io/) (recommended)
 
@@ -91,7 +102,7 @@ echo {}> .prettierrc.json
 
 **Note:** Please read some important instructions regarding the `project` option [here](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/parser/README.md#configuration).
 
-You must have a valid `tsconfig.json` file to specify `"parserOptions.project"`.
+`"parserOptions.project"` is only required if you use **TypeScript**.
 
 #### `package.json`
 

eslintrc.json

@@ -0,0 +1,6 @@
+{
+  "extends": "./index.js",
+  "parserOptions": {
+    "project": "./tsconfig.json"
+  }
+}

package.json

@@ -31,10 +31,10 @@
     "lint:commit": "commitlint",
     "lint:editorconfig": "editorconfig-checker",
     "lint:markdown": "markdownlint \"**/*.md\" --dot --ignore-path \".gitignore\"",
-    "lint:javascript": "eslint \"**/*.{js,jsx,ts,tsx}\"",
+    "lint:javascript": "eslint \"**/*.{js,jsx,ts,tsx}\" -c \"eslintrc.json\"",
     "lint:prettier": "prettier \".\" --check --ignore-path \".gitignore\"",
     "lint:staged": "lint-staged",
-    "test": "tape \"test/**/*.js\"",
+    "test": "tap",
     "release": "semantic-release",
     "postinstall": "husky install",
     "prepublishOnly": "pinst --disable",
@@ -42,31 +42,29 @@
   },
   "peerDependencies": {
     "eslint": "^8.9.0",
-    "eslint-plugin-import": "^2.25.4",
+    "eslint-plugin-import": "^2.26.0",
     "eslint-plugin-promise": "^6.0.0",
-    "eslint-plugin-unicorn": "^41.0.0",
-    "typescript": "^4.4.0",
-    "@typescript-eslint/eslint-plugin": "^5.12.0"
+    "eslint-plugin-unicorn": "^42.0.0"
   },
   "devDependencies": {
-    "@commitlint/cli": "16.2.1",
+    "@commitlint/cli": "16.2.3",
     "@commitlint/config-conventional": "16.2.1",
-    "editorconfig-checker": "4.0.2",
-    "eslint": "8.9.0",
     "@types/eslint": "8.4.1",
-    "eslint-plugin-import": "2.25.4",
+    "@types/tap": "15.0.6",
+    "@typescript-eslint/eslint-plugin": "5.18.0",
+    "@typescript-eslint/parser": "5.18.0",
+    "editorconfig-checker": "4.0.2",
+    "eslint": "8.12.0",
+    "eslint-plugin-import": "2.26.0",
     "eslint-plugin-promise": "6.0.0",
-    "eslint-plugin-unicorn": "41.0.0",
-    "typescript": "4.5.5",
-    "@typescript-eslint/parser": "5.12.0",
-    "@typescript-eslint/eslint-plugin": "5.12.0",
-    "tape": "5.5.2",
+    "eslint-plugin-unicorn": "42.0.0",
     "husky": "7.0.4",
-    "lint-staged": "12.3.4",
+    "lint-staged": "12.3.7",
     "markdownlint-cli": "0.31.1",
-    "@types/tape": "4.13.2",
-    "pinst": "2.1.6",
-    "prettier": "2.5.1",
-    "semantic-release": "19.0.2"
+    "pinst": "3.0.0",
+    "prettier": "2.6.2",
+    "semantic-release": "19.0.2",
+    "tap": "16.0.1",
+    "typescript": "4.6.3"
   }
 }

test/basic.js

@@ -1,13 +1,12 @@
-const test = require('tape')
+const tap = require('tap')
 
 const config = require('../index.js')
 
-test('test basic properties of config', function (t) {
+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))
-  t.end()
 })
 
 function isObject(object) {

test/validate-config.js

@@ -1,8 +1,13 @@
 const { ESLint } = require('eslint')
-const test = require('tape')
+const tap = require('tap')
 
-test('ensure we validate correctly JavaScript files', async (t) => {
-  const eslint = new ESLint({ ignore: false })
+const eslint = new ESLint({
+  ignore: false,
+  useEslintrc: false,
+  overrideConfigFile: 'eslintrc.json'
+})
+
+tap.test('ensure we validate correctly JavaScript files', async (t) => {
   const [noErrors] = await eslint.lintFiles(
     'test/fixtures/javascript-no-errors.js'
   )
@@ -11,11 +16,9 @@ test('ensure we validate correctly JavaScript files', async (t) => {
   )
   t.equal(noErrors.errorCount, 0)
   t.equal(withErrors.errorCount, 3)
-  t.end()
 })
 
-test('ensure we validate correctly TypeScript files', async (t) => {
-  const eslint = new ESLint({ ignore: false })
+tap.test('ensure we validate correctly TypeScript files', async (t) => {
   const [noErrors] = await eslint.lintFiles(
     'test/fixtures/typescript-no-errors.ts'
   )
@@ -24,14 +27,11 @@ test('ensure we validate correctly TypeScript files', async (t) => {
   )
   t.equal(noErrors.errorCount, 0)
   t.equal(withErrors.errorCount, 3)
-  t.end()
 })
 
-test('ensure we allow top-level await', async (t) => {
-  const eslint = new ESLint({ ignore: false })
+tap.test('ensure we allow top-level await', async (t) => {
   const [lintResult] = await eslint.lintFiles(
     'test/fixtures/top-level-await.mjs'
   )
   t.equal(lintResult.errorCount, 0)
-  t.end()
 })