Merge pull request #157 from remarkablemark/build/devDependencies

build: upgrade devDependencies

Author: remarkablemark

.eslintrc renamed to .eslintrc.json

@@ -12,6 +12,8 @@
   },
   "plugins": ["prettier"],
   "rules": {
+    "no-debugger": "error",
+    "no-console": "error",
     "prettier/prettier": "error"
   }
 }

.huskyrc

@@ -0,0 +1,6 @@
+{
+  "hooks": {
+    "commit-msg": "commitlint -E HUSKY_GIT_PARAMS",
+    "pre-commit": "npm run lint:dts && npm run test:coverage && lint-staged"
+  }
+}

.huskyrc.json

@@ -0,0 +1,4 @@
+{
+  "*.js": "npm run lint:fix",
+  "*.{html,json,md,yml}": "prettier --write"
+}

.lintstagedrc

@@ -0,0 +1,5 @@
+{
+  "arrowParens": "avoid",
+  "trailingComma": "none",
+  "singleQuote": true
+}

.lintstagedrc.json

@@ -7,10 +7,10 @@ var domParserOptions = { decodeEntities: true, lowerCaseAttributeNames: false };
 /**
  * Converts HTML string to React elements.
  *
- * @param  {String}   html              - The HTML string to parse to React.
- * @param  {Object}   [options]         - The parser options.
- * @param  {Function} [options.replace] - The replace method.
- * @return {ReactElement|Array|String}  - When parsed with HTML string, returns React elements; otherwise, returns string or empty array.
+ * @param  {String}   html                    - The HTML string to parse to React.
+ * @param  {Object}   [options]               - The parser options.
+ * @param  {Function} [options.replace]       - The replace method.
+ * @return {JSX.Element|JSX.Element[]|String} - Returns React element(s), string, or empty array.
  */
 function HTMLReactParser(html, options) {
   if (typeof html !== 'string') {
@@ -22,5 +22,6 @@ function HTMLReactParser(html, options) {
 HTMLReactParser.domToReact = domToReact;
 HTMLReactParser.htmlToDOM = htmlToDOM;
 
+// support CommonJS and ES Modules
 module.exports = HTMLReactParser;
 module.exports.default = HTMLReactParser;

.prettierrc

@@ -77,7 +77,7 @@ function cssToJs(style) {
   var styleObject = {};
 
   if (style) {
-    styleToObject(style, function(property, value) {
+    styleToObject(style, function (property, value) {
       // skip CSS comment
       if (property && value) {
         styleObject[camelCase(property)] = value;

.prettierrc.json

@@ -21,7 +21,7 @@ function camelCase(string) {
   // convert to camelCase
   return string
     .toLowerCase()
-    .replace(hyphenPatternRegex, function(_, character) {
+    .replace(hyphenPatternRegex, function (_, character) {
       return character.toUpperCase();
     });
 }

index.js

@@ -6,14 +6,12 @@
   "main": "index.js",
   "scripts": {
     "benchmark": "node benchmark",
-    "build": "npm run clean && npm run build:min && npm run build:unmin",
-    "build:min": "cross-env NODE_ENV=production rollup --config --file dist/html-react-parser.min.js --sourcemap",
-    "build:unmin": "cross-env NODE_ENV=development rollup --config --file dist/html-react-parser.js",
+    "build": "rollup --config",
     "clean": "rimraf dist",
     "lint": "eslint --ignore-path .gitignore --ignore-pattern /examples/ .",
     "lint:dts": "dtslint .",
     "lint:fix": "npm run lint -- --fix",
-    "prepublishOnly": "npm run lint && npm run lint:dts && npm test && npm run build",
+    "prepublishOnly": "npm run lint && npm run lint:dts && npm test && npm run clean && npm run build",
     "release": "standard-version --no-verify",
     "test": "mocha",
     "test:coverage": "nyc npm test",
@@ -40,28 +38,28 @@
     "style-to-object": "0.3.0"
   },
   "devDependencies": {
-    "@commitlint/cli": "^8.2.0",
-    "@commitlint/config-conventional": "^8.2.0",
-    "@types/react": "^16.9.11",
+    "@commitlint/cli": "^8.3.5",
+    "@commitlint/config-conventional": "^8.3.4",
+    "@types/react": "^16.9.35",
     "benchmark": "^2.1.4",
-    "cross-env": "^6.0.3",
-    "dtslint": "^2.0.0",
-    "eslint": "^6.6.0",
-    "eslint-plugin-prettier": "^3.1.1",
-    "husky": "^3.0.9",
-    "lint-staged": "^9.4.2",
-    "mocha": "^6.2.2",
-    "nyc": "^14.1.1",
-    "preact": "^10.0.4",
-    "prettier": "^1.19.1",
+    "dtslint": "^3.6.4",
+    "eslint": "^7.1.0",
+    "eslint-plugin-prettier": "^3.1.3",
+    "husky": "^4.2.5",
+    "lint-staged": "^10.2.6",
+    "mocha": "^7.2.0",
+    "nyc": "^15.0.1",
+    "preact": "^10.4.4",
+    "prettier": "^2.0.5",
     "react": "^16",
     "react-dom": "^16",
-    "rimraf": "^3.0.0",
-    "rollup": "^1.26.3",
+    "rimraf": "^3.0.2",
+    "rollup": "^1.32.1",
     "rollup-plugin-commonjs": "^10.1.0",
     "rollup-plugin-node-resolve": "^5.2.0",
-    "rollup-plugin-uglify": "^6.0.3",
-    "standard-version": "^6"
+    "rollup-plugin-uglify": "^6.0.4",
+    "standard-version": "^6",
+    "typescript": "^3.9.3"
   },
   "peerDependencies": {
     "react": "^0.14 || ^15 || ^16"

lib/attributes-to-props.js

@@ -2,21 +2,25 @@ import commonjs from 'rollup-plugin-commonjs';
 import resolve from 'rollup-plugin-node-resolve';
 import { uglify } from 'rollup-plugin-uglify';
 
-const config = {
+/**
+ * Build rollup config for development (default) or production (minify = true).
+ *
+ * @param {Boolean} [minify=false]
+ * @return {Object}
+ */
+const getConfig = (minify = false) => ({
   external: ['react'],
   input: 'index.js',
   output: {
+    file: `dist/html-react-parser${minify ? '.min' : ''}.js`,
     format: 'umd',
     globals: {
       react: 'React'
     },
-    name: 'HTMLReactParser'
+    name: 'HTMLReactParser',
+    sourcemap: true
   },
-  plugins: [commonjs(), resolve({ browser: true })]
-};
+  plugins: [commonjs(), resolve({ browser: true }), minify && uglify()]
+});
 
-if (process.env.NODE_ENV === 'production') {
-  config.plugins.push(uglify());
-}
-
-export default config;
+export default [getConfig(), getConfig(true)];

lib/utilities.js

@@ -1,9 +1,23 @@
 const assert = require('assert');
 const React = require('react');
-const parse = require('../');
+const parse = require('..');
 const { data, render } = require('./helpers/');
 
 describe('html-to-react', () => {
+  describe('exports', () => {
+    it('has default ES Module', () => {
+      assert.strictEqual(parse.default, parse);
+    });
+
+    it('has domToReact', () => {
+      assert.strictEqual(parse.domToReact, require('../lib/dom-to-react'));
+    });
+
+    it('contains htmlToDOM', () => {
+      assert.strictEqual(parse.htmlToDOM, require('html-dom-parser'));
+    });
+  });
+
   describe('parser', () => {
     [undefined, null, {}, [], 42].forEach(value => {
       it(`throws an error if first argument is ${value}`, () => {
@@ -13,10 +27,6 @@ describe('html-to-react', () => {
       });
     });
 
-    it('has default ES Module defined', () => {
-      assert.strictEqual(parse.default, parse);
-    });
-
     it('returns string if it cannot be parsed as HTML', () => {
       assert.strictEqual(parse('foo'), 'foo');
     });
@@ -77,7 +87,7 @@ describe('html-to-react', () => {
 
   describe('options', () => {
     describe('replace', () => {
-      it('overrides the element if replace is valid', () => {
+      it('overrides the element if a valid React element is returned', () => {
         const html = data.html.complex;
         const reactElement = parse(html, {
           replace: node => {
@@ -92,7 +102,7 @@ describe('html-to-react', () => {
         );
       });
 
-      it('does not override the element if replace is invalid', () => {
+      it('does not override the element if an invalid React element is returned', () => {
         const html = data.html.complex;
         const reactElement = parse(html, {
           replace: node => {
@@ -115,9 +125,3 @@ describe('html-to-react', () => {
     });
   });
 });
-
-describe('dom-to-react', () => {
-  it('exports domToReact', () => {
-    assert.strictEqual(parse.domToReact, require('../lib/dom-to-react'));
-  });
-});