Merge pull request #435 from ekalinin/next2

Next

Author: derduher

Diff for: .eslintrc.js

@@ -0,0 +1,78 @@
+// eslint.config.js
+module.exports = {
+  env: {
+    es6: true,
+    jest: true,
+    node: true,
+  },
+  parser: '@typescript-eslint/parser',
+  parserOptions: {
+    ecmaVersion: 2023,
+    sourceType: 'module',
+  },
+  plugins: ['jest', '@typescript-eslint'],
+  extends: [
+    'eslint:recommended',
+    'plugin:@typescript-eslint/eslint-recommended',
+    'plugin:@typescript-eslint/recommended',
+    'prettier',
+    'plugin:prettier/recommended',
+  ],
+  rules: {
+    indent: 'off',
+    'lines-between-class-members': [
+      'error',
+      'always',
+      {
+        exceptAfterSingleLine: true,
+      },
+    ],
+    'no-case-declarations': 0,
+    'no-console': 0,
+    'no-dupe-class-members': 'off',
+    'no-unused-vars': 0,
+    'padding-line-between-statements': [
+      'error',
+      {
+        blankLine: 'always',
+        prev: 'multiline-expression',
+        next: 'multiline-expression',
+      },
+    ],
+    '@typescript-eslint/ban-ts-comment': [
+      'error',
+      {
+        'ts-expect-error': 'allow-with-description',
+      },
+    ],
+    '@typescript-eslint/explicit-member-accessibility': 'off',
+    '@typescript-eslint/naming-convention': [
+      'error',
+      {
+        selector: 'default',
+        format: null,
+      },
+      {
+        selector: 'interface',
+        prefix: [],
+        format: null,
+      },
+    ],
+    '@typescript-eslint/no-parameter-properties': 'off',
+    '@typescript-eslint/no-unused-vars': [
+      'error',
+      {
+        args: 'none',
+      },
+    ],
+  },
+  overrides: [
+    {
+      files: ['*.js'],
+      rules: {
+        '@typescript-eslint/explicit-function-return-type': ['off'],
+        '@typescript-eslint/no-var-requires': ['off'],
+      },
+    },
+  ],
+};

Diff for: .github/workflows/nodejs.yml

@@ -23,7 +23,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node-version: [14.x, 16.x]
+        node-version: [18.x, 20.x]
     steps:
     - uses: actions/checkout@v4
     - uses: ./.github/actions/configure-nodejs

Diff for: .husky/pre-commit

@@ -1,4 +1 @@
-#!/bin/sh
-. "$(dirname "$0")/_/husky.sh"
-
 npx --no-install lint-staged && npm run test

Diff for: CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 9.0.0
+
+- dropped support for all node versions less than 18
+
 ## 8.0.0
 
 - fix #423 via #424 thanks @huntharo - Propagate errors in SitemapAndIndexStream

Diff for: babel.config.js

@@ -1,9 +1,5 @@
 module.exports = {
-  plugins: [
-    '@babel/plugin-proposal-class-properties',
-    '@babel/plugin-proposal-optional-chaining',
-    '@babel/plugin-proposal-nullish-coalescing-operator',
-  ],
+  plugins: [],
   presets: [
     ['@babel/preset-env', { targets: { node: 'current' } }],
     '@babel/preset-typescript',

Diff for: jest.config.js

@@ -0,0 +1,20 @@
+/** @type {import('jest').Config} */
+const config = {
+  collectCoverage: true,
+  collectCoverageFrom: [
+    'lib/**/*.ts',
+    '!lib/**/*.d.ts',
+    '!lib/xmllint.ts',
+    '!node_modules/',
+  ],
+  coverageThreshold: {
+    global: {
+      branches: 80,
+      functions: 90,
+      lines: 90,
+      statements: 90,
+    },
+  },
+};
+
+module.exports = config;