Change coding convention

Author: ndaidong

Diff for: .eslintignore

@@ -0,0 +1,2 @@
+node_modules
+dist

Diff for: .eslintrc.json

@@ -0,0 +1,121 @@
+{
+  "parserOptions": {
+    "ecmaVersion": "latest",
+    "sourceType": "module"
+  },
+  "env": {
+    "es6": true,
+    "node": true,
+    "browser": true,
+    "jest": true
+  },
+  "globals": {
+    "globalThis": true
+  },
+  "plugins": [],
+  "overrides": [],
+  "extends": ["eslint:recommended"],
+  "rules": {
+    "arrow-spacing": ["error", { "before": true, "after": true }],
+    "block-spacing": ["error", "always"],
+    "brace-style": ["error", "1tbs", { "allowSingleLine": true }],
+    "camelcase": ["error", {
+      "allow": ["^UNSAFE_"],
+      "properties": "never",
+      "ignoreGlobals": true
+    }],
+    "comma-dangle": ["error", {
+      "arrays": "always-multiline",
+      "objects": "always-multiline",
+      "imports": "never",
+      "exports": "never",
+      "functions": "never"
+    }],
+    "comma-spacing": ["error", { "before": false, "after": true }],
+    "eol-last": "error",
+    "eqeqeq": ["error", "always", { "null": "ignore" }],
+    "func-call-spacing": ["error", "never"],
+    "indent": [
+      "error",
+      2,
+      {
+        "MemberExpression": 1,
+        "FunctionDeclaration": {
+          "body": 1,
+          "parameters": 2
+        },
+        "SwitchCase": 1
+      }
+    ],
+    "key-spacing": ["error", { "beforeColon": false, "afterColon": true }],
+    "keyword-spacing": ["error", { "before": true, "after": true }],
+    "lines-between-class-members": ["error", "always", { "exceptAfterSingleLine": true }],
+    "max-len": [
+      "error",
+      {
+        "code": 120,
+        "ignoreTrailingComments": true,
+        "ignoreComments": true,
+        "ignoreUrls": true
+      }
+    ],
+    "max-lines": [
+      "error",
+      {
+        "max": 360,
+        "skipBlankLines": true,
+        "skipComments": false
+      }
+    ],
+    "max-lines-per-function": [
+      "error",
+      {
+        "max": 150,
+        "skipBlankLines": true
+      }
+    ],
+    "max-params": ["error", 3],
+    "no-array-constructor": "error",
+    "no-mixed-spaces-and-tabs": "error",
+    "no-multi-spaces": "error",
+    "no-multi-str": "error",
+    "no-multiple-empty-lines": [
+      "error",
+      {
+        "max": 1,
+        "maxEOF": 0
+      }
+    ],
+    "no-restricted-syntax": [
+        "error",
+        "WithStatement",
+        "BinaryExpression[operator='in']"
+    ],
+    "no-trailing-spaces": "error",
+    "no-use-before-define": [
+      "error",
+      {
+        "functions": true,
+        "classes": true,
+        "variables": false
+      }
+    ],
+    "no-var": "warn",
+    "object-curly-spacing": ["error", "always"],
+    "padded-blocks": [
+      "error",
+      {
+        "blocks": "never",
+        "switches": "never",
+        "classes": "never"
+      }
+    ],
+    "quotes": ["error", "single"],
+    "space-before-blocks": ["error", "always"],
+    "space-before-function-paren": ["error", "always"],
+    "space-infix-ops": "error",
+    "space-unary-ops": ["error", { "words": true, "nonwords": false }],
+    "space-in-parens": ["error", "never"],
+    "semi": ["error", "never"]
+  }
+}

Diff for: .npmignore

@@ -0,0 +1,6 @@
+node_modules
+coverage
+.github
+pnpm-lock.yaml
+examples
+test-data

Diff for: CONTRIBUTING.md

@@ -16,11 +16,8 @@ This library needs to be simple and flexible to run on multiple platforms such a
 
 ## Coding convention
 
-Please follow [standardjs](https://standardjs.com/) style guide.
-
 Make sure your code lints before opening a pull request.
 
-
 ```bash
 cd feed-extractor
 

Diff for: README.md

@@ -6,12 +6,9 @@ To read & normalize RSS/ATOM/JSON feed data.
 ![CodeQL](https://github.com/extractus/feed-extractor/workflows/CodeQL/badge.svg)
 ![CI test](https://github.com/extractus/feed-extractor/workflows/ci-test/badge.svg)
 [![Coverage Status](https://img.shields.io/coveralls/github/extractus/feed-extractor)](https://coveralls.io/github/extractus/feed-extractor?branch=main)
-[![JavaScript Style Guide](https://img.shields.io/badge/code_style-standard-brightgreen.svg)](https://standardjs.com)
-
-### Attention
-
-`feed-reader` has been renamed to `@extractus/feed-extractor` since v6.1.4
+[![CodeFactor](https://www.codefactor.io/repository/github/extractus/feed-extractor/badge)](https://www.codefactor.io/repository/github/extractus/feed-extractor)
 
+(This library is derived from [feed-reader](https://www.npmjs.com/package/feed-reader) renamed.)
 
 ## Demo
 

Diff for: build.js

@@ -8,15 +8,15 @@ const pkg = JSON.parse(readFileSync('./package.json', { encoding: 'utf-8' }))
 
 rmSync('dist', {
   force: true,
-  recursive: true
+  recursive: true,
 })
 mkdirSync('dist')
 
 const buildTime = (new Date()).toISOString()
 const comment = [
   `// ${pkg.name}@${pkg.version}, by ${pkg.author}`,
   `built with esbuild at ${buildTime}`,
-  `published under ${pkg.license} license`
+  `published under ${pkg.license} license`,
 ].join(' - ')
 
 const baseOpt = {
@@ -28,7 +28,7 @@ const baseOpt = {
   legalComments: 'none',
   minify: false,
   sourcemap: false,
-  write: true
+  write: true,
 }
 
 const esmVersion = {
@@ -37,8 +37,8 @@ const esmVersion = {
   format: 'esm',
   outfile: 'dist/feed-extractor.esm.js',
   banner: {
-    js: comment
-  }
+    js: comment,
+  },
 }
 buildSync(esmVersion)
 
@@ -49,15 +49,15 @@ const cjsVersion = {
   mainFields: ['main'],
   outfile: 'dist/cjs/feed-extractor.js',
   banner: {
-    js: comment
-  }
+    js: comment,
+  },
 }
 buildSync(cjsVersion)
 
 const cjspkg = {
   name: pkg.name,
   version: pkg.version,
-  main: './feed-extractor.js'
+  main: './feed-extractor.js',
 }
 
 writeFileSync(

Diff for: examples/browser-feed-reader/server.js

@@ -8,7 +8,7 @@ const app = express()
 const loadRemoteFeed = async (url) => {
   try {
     const headers = {
-      'Accept-Charset': 'utf-8'
+      'Accept-Charset': 'utf-8',
     }
     const data = await got(url, { headers }).text()
     return data

Diff for: examples/node-feed-reader/index.js

@@ -7,7 +7,7 @@ const meta = {
   service: 'feed-reader',
   lang: 'javascript',
   server: 'express',
-  platform: 'node'
+  platform: 'node',
 }
 
 app.get('/', async (req, res) => {
@@ -20,14 +20,14 @@ app.get('/', async (req, res) => {
     includeEntryContent = 'n',
     includeOptionalElements = 'n',
     useISODateFormat = 'y',
-    normalization = 'y'
+    normalization = 'y',
   } = req.query
 
   const opts = {
     includeEntryContent: includeEntryContent === 'y',
     includeOptionalElements: includeOptionalElements === 'y',
     useISODateFormat: useISODateFormat !== 'n',
-    normalization: normalization !== 'n'
+    normalization: normalization !== 'n',
   }
 
   try {
@@ -36,14 +36,14 @@ app.get('/', async (req, res) => {
       error: 0,
       message: 'feed data has been extracted successfully',
       data,
-      meta
+      meta,
     })
   } catch (err) {
     return res.json({
       error: 1,
       message: err.message,
       data: null,
-      meta
+      meta,
     })
   }
 })

Diff for: package.json

@@ -25,8 +25,8 @@
     "node": ">= 14"
   },
   "scripts": {
-    "lint": "standard .",
-    "lint:fix": "standard --fix",
+    "lint": "eslint .",
+    "lint:fix": "eslint --fix .",
     "pretest": "npm run lint",
     "test": "NODE_ENV=test NODE_OPTIONS=--experimental-vm-modules jest --verbose --coverage=true",
     "eval": "node eval",
@@ -36,20 +36,14 @@
   "dependencies": {
     "bellajs": "^11.1.1",
     "cross-fetch": "^3.1.5",
-    "fast-xml-parser": "^4.0.12",
+    "fast-xml-parser": "^4.0.13",
     "html-entities": "^2.3.3"
   },
   "devDependencies": {
-    "esbuild": "^0.16.13",
+    "esbuild": "^0.16.17",
+    "eslint": "^8.31.0",
     "jest": "^29.3.1",
-    "nock": "^13.2.9",
-    "standard": "^17.0.0"
-  },
-  "standard": {
-    "ignore": [
-      "/dist",
-      "/examples"
-    ]
+    "nock": "^13.3.0"
   },
   "keywords": [
     "extractor",

Diff for: reset.js

@@ -13,14 +13,14 @@ const dirs = [
   '.nyc_output',
   'coverage',
   'node_modules',
-  '.nuxt'
+  '.nuxt',
 ]
 
 const files = [
   'yarn.lock',
   'pnpm-lock.yaml',
   'package-lock.json',
-  'coverage.lcov'
+  'coverage.lcov',
 ]
 
 dirs.forEach((d) => {

Diff for: src/main.js

@@ -25,15 +25,15 @@ export const read = async (url, options = {}, fetchOptions = {}) => {
     useISODateFormat = true,
     xmlParserOptions = {},
     getExtraFeedFields = () => ({}),
-    getExtraEntryFields = () => ({})
+    getExtraEntryFields = () => ({}),
   } = options
 
   const opts = {
     normalization,
     descriptionMaxLen,
     useISODateFormat,
     getExtraFeedFields,
-    getExtraEntryFields
+    getExtraEntryFields,
   }
 
   if (type === 'json') {