Refactor code-style

Author: wooorm

.babelrc

@@ -1,5 +1,3 @@
 {
-  "presets": [
-    ["@babel/env"]
-  ]
+  "presets": [["@babel/env"]]
 }

.eslintignore

@@ -0,0 +1,4 @@
+coverage/
+dist/
+src/__fixtures__/**/*.html
+*.md

.eslintrc.js

@@ -1,33 +1,31 @@
-const babel = require('@rollup/plugin-babel').default;
-const commonjs = require('@rollup/plugin-commonjs');
-const json = require('@rollup/plugin-json');
-const resolve = require('@rollup/plugin-node-resolve').default;
+const babel = require('@rollup/plugin-babel').default
+const commonjs = require('@rollup/plugin-commonjs')
+const json = require('@rollup/plugin-json')
+const resolve = require('@rollup/plugin-node-resolve').default
 
-module.exports = function karmaConfig(config) {
+module.exports = karmaConfig
+
+function karmaConfig(config) {
   config.set({
     frameworks: ['jasmine'],
-    files: [
-      { pattern: 'src/index.test.js' },
-    ],
+    files: [{pattern: 'src/index.test.js'}],
     preprocessors: {
-      'src/*.test.js': ['rollup'],
+      'src/*.test.js': ['rollup']
     },
     rollupPreprocessor: {
-      output: { format: 'iife' },
+      output: {format: 'iife'},
       plugins: [
         resolve({
           browser: true,
-          extensions: ['.mjs', '.js', '.json', '.node'],
+          extensions: ['.mjs', '.js', '.json', '.node']
         }),
         json(),
         commonjs(),
         babel({
-          presets: [
-            ['@babel/preset-env', { modules: false }],
-          ],
-          babelrc: false,
-        }),
-      ],
-    },
-  });
-};
+          presets: [['@babel/preset-env', {modules: false}]],
+          babelrc: false
+        })
+      ]
+    }
+  })
+}

.prettierignore

@@ -41,11 +41,7 @@
     "@rollup/plugin-commonjs": "^18.0.0",
     "@rollup/plugin-json": "^4.0.0",
     "@rollup/plugin-node-resolve": "^11.0.0",
-    "babel-eslint": "^10.0.0",
     "babel-jest": "^26.0.0",
-    "eslint": "^7.0.0",
-    "eslint-config-airbnb-base": "^14.0.0",
-    "eslint-plugin-import": "^2.0.0",
     "glob": "^7.0.0",
     "hastscript": "^6.0.0",
     "jasmine-core": "^3.0.0",
@@ -57,15 +53,17 @@
     "karma-mocha-reporter": "^2.0.0",
     "karma-rollup-preprocessor": "^7.0.0",
     "karma-safari-launcher": "^1.0.0",
+    "prettier": "^2.0.0",
     "remark-cli": "^9.0.0",
     "remark-preset-wooorm": "^8.0.0",
     "rollup": "^2.0.0",
-    "w3c-xmlserializer": "^2.0.0"
+    "w3c-xmlserializer": "^2.0.0",
+    "xo": "^0.38.0"
   },
   "scripts": {
     "clean": "rm -rf dist",
     "build": "rollup -c",
-    "format": "remark . -qfo && eslint . --fix",
+    "format": "remark . -qfo && prettier . -w --loglevel warn && xo --fix",
     "test:api": "jest",
     "test:karma": "karma start --single-run --browsers ChromeHeadless,FirefoxHeadless,Safari",
     "test:karma:chrome": "karma start --single-run --browsers ChromeHeadless",
@@ -74,6 +72,17 @@
     "test:dev": "jest --watchAll",
     "test": "yarn build && yarn format && yarn test:api"
   },
+  "prettier": {
+    "tabWidth": 2,
+    "useTabs": false,
+    "singleQuote": true,
+    "bracketSpacing": false,
+    "semi": false,
+    "trailingComma": "none"
+  },
+  "xo": {
+    "prettier": true
+  },
   "jest": {
     "collectCoverage": true,
     "coveragePathIgnorePatterns": [

karma.conf.js

@@ -1,25 +1,25 @@
-import babel from '@rollup/plugin-babel';
+import babel from '@rollup/plugin-babel'
 
-import pkg from './package.json';
+import pkg from './package.json'
 
-export default {
+const config = {
   input: 'src/index.js',
   output: [
-    { file: pkg.main, format: 'cjs', exports: 'default' },
-    { file: pkg.module, format: 'es' },
+    {file: pkg.main, format: 'cjs', exports: 'default'},
+    {file: pkg.module, format: 'es'}
   ],
   plugins: [
     babel({
-      presets: [
-        ['@babel/preset-env', { modules: false }],
-      ],
-      babelrc: false,
-    }),
+      presets: [['@babel/preset-env', {modules: false}]],
+      babelrc: false
+    })
   ],
   external: [
     'property-information/find',
     'property-information/html',
     'property-information/svg',
-    'web-namespaces',
-  ],
-};
+    'web-namespaces'
+  ]
+}
+
+export default config

package.json

@@ -7,16 +7,9 @@
       "type": "element",
       "tagName": "svg",
       "properties": {
-        "className": [
-          "icon"
-        ],
+        "className": ["icon"],
         "ariaLabel": "Verified",
-        "viewBox": [
-          0,
-          0,
-          16,
-          16
-        ],
+        "viewBox": [0, 0, 16, 16],
         "width": 18,
         "height": 18,
         "role": "img"

rollup.config.js

@@ -58,4 +58,4 @@
       "value": "\n"
     }
   ]
-}
+}

src/__fixtures__/svg-advanced/index.json

@@ -1,33 +1,39 @@
-import fs from 'fs';
-import path from 'path';
-import glob from 'glob';
+/* eslint-env jest, browser */
 
-import serializeNodeToHtmlString from './utils';
-import toDOM from './index';
+import fs from 'fs'
+import path from 'path'
+import glob from 'glob'
+import serializeNodeToHtmlString from './utils.js'
+import toDOM from './index.js'
 
 describe('fixtures', () => {
-  const root = path.join(__dirname, '__fixtures__');
-  const fixturePaths = glob.sync(path.join(root, '**/*/'));
+  const root = path.join(__dirname, '__fixtures__')
+  const fixturePaths = glob.sync(path.join(root, '**/*/'))
+  let index = -1
 
-  fixturePaths.forEach((fixturePath) => {
-    const fixture = path.relative(root, fixturePath);
-    const fixtureInput = path.join(fixturePath, 'index.json');
-    const fixtureOutput = path.join(fixturePath, 'index.html');
+  while (++index < fixturePaths.length) {
+    each(fixturePaths[index])
+  }
+
+  function each(fixturePath) {
+    const fixture = path.relative(root, fixturePath)
+    const fixtureInput = path.join(fixturePath, 'index.json')
+    const fixtureOutput = path.join(fixturePath, 'index.html')
 
     test(fixture, () => {
-      const fixtureData = JSON.parse(fs.readFileSync(fixtureInput));
-      const parsedActual = serializeNodeToHtmlString(toDOM(fixtureData));
+      const fixtureData = JSON.parse(fs.readFileSync(fixtureInput))
+      const parsedActual = serializeNodeToHtmlString(toDOM(fixtureData))
 
-      let parsedExpected;
+      let parsedExpected
 
       try {
-        parsedExpected = fs.readFileSync(fixtureOutput).toString().trim();
-      } catch (e) {
-        fs.writeFileSync(fixtureOutput, parsedActual);
-        return;
+        parsedExpected = fs.readFileSync(fixtureOutput).toString().trim()
+      } catch {
+        fs.writeFileSync(fixtureOutput, parsedActual)
+        return
       }
 
-      expect(parsedActual).toEqual(parsedExpected);
-    });
-  });
-});
+      expect(parsedActual).toEqual(parsedExpected)
+    })
+  }
+})