Refactor code-style

Author: wooorm

.prettierignore

@@ -0,0 +1,3 @@
+coverage/
+mdast-util-to-string.js
+mdast-util-to-string.min.js

index.js

@@ -1,22 +1,22 @@
-'use strict';
+'use strict'
 
-/* Expose. */
-module.exports = toString;
+module.exports = toString
 
 /* Get the text content of a node.  If the node itself
  * does not expose plain-text fields, `toString` will
  * recursivly try its children. */
 function toString(node) {
-  return valueOf(node) ||
+  return (
+    valueOf(node) ||
     (node.children && node.children.map(toString).join('')) ||
-    '';
+    ''
+  )
 }
 
 /* Get the value of `node`.  Checks, `value`,
  * `alt`, and `title`, in that order. */
 function valueOf(node) {
-  if (!node) {
-    return '';
-  }
-  return node.value ? node.value : (node.alt ? node.alt : node.title) || '';
+  return (
+    (node && node.value ? node.value : node.alt ? node.alt : node.title) || ''
+  )
 }

package.json

@@ -25,24 +25,32 @@
   "devDependencies": {
     "browserify": "^16.0.0",
     "nyc": "^12.0.0",
+    "prettier": "^1.14.2",
     "remark-cli": "^5.0.0",
     "remark-preset-wooorm": "^4.0.0",
     "tape": "^4.4.0",
     "tinyify": "^2.4.3",
     "xo": "^0.22.0"
   },
   "scripts": {
-    "build-md": "remark . --quiet --frail --output",
+    "format": "remark . -qfo && prettier --write '**/*.js' && xo --fix",
     "build-bundle": "browserify . -s mdastUtilToString > mdast-util-to-string.js",
     "build-mangle": "browserify . -s mdastUtilToString -p tinyify > mdast-util-to-string.min.js",
-    "build": "npm run build-md && npm run build-bundle && npm run build-mangle",
-    "lint": "xo",
-    "test-api": "node test.js",
+    "build": "npm run build-bundle && npm run build-mangle",
+    "test-api": "node test",
     "test-coverage": "nyc --reporter lcov tape test.js",
-    "test": "npm run build && npm run lint && npm run test-coverage"
+    "test": "npm run format && npm run build && npm run test-coverage"
+  },
+  "prettier": {
+    "tabWidth": 2,
+    "useTabs": false,
+    "singleQuote": true,
+    "bracketSpacing": false,
+    "semi": false,
+    "trailingComma": "none"
   },
   "xo": {
-    "space": true,
+    "prettier": true,
     "esnext": false,
     "ignore": [
       "mdast-util-to-string.js"

readme.md

@@ -13,15 +13,15 @@ npm install mdast-util-to-string
 ## Usage
 
 ```js
-var unified = require('unified');
-var parse = require('remark-parse');
-var toString = require('mdast-util-to-string');
+var unified = require('unified')
+var parse = require('remark-parse')
+var toString = require('mdast-util-to-string')
 
 var tree = unified()
   .use(parse)
-  .parse('Some _emphasis_, **importance**, and `code`.');
+  .parse('Some _emphasis_, **importance**, and `code`.')
 
-console.log(toString(tree)); //=> 'Some emphasis, importance, and code.'
+console.log(toString(tree)) // => 'Some emphasis, importance, and code.'
 ```
 
 ## API

test.js

@@ -1,21 +1,18 @@
-'use strict';
+'use strict'
 
-var test = require('tape');
-var toString = require('.');
+var test = require('tape')
+var toString = require('.')
 
-test('mdast-util-to-string', function (t) {
-  t.throws(
-    function () {
-      toString();
-    },
-    'should fail without node'
-  );
+test('mdast-util-to-string', function(t) {
+  t.throws(function() {
+    toString()
+  }, 'should fail without node')
 
   t.equal(
     toString({value: 'foo'}),
     'foo',
     'should not fail on unrecognised nodes'
-  );
+  )
 
   t.equal(
     toString({
@@ -24,23 +21,19 @@ test('mdast-util-to-string', function (t) {
     }),
     'foo',
     'should prefer `value` over all others'
-  );
+  )
 
   t.equal(
-    toString({
-      value: 'foo',
-      alt: 'bar',
-      title: 'baz'
-    }),
+    toString({value: 'foo', alt: 'bar', title: 'baz'}),
     'foo',
     'should prefer `value` over `alt` or `title`'
-  );
+  )
 
   t.equal(
     toString({alt: 'bar', title: 'baz'}),
     'bar',
     'should prefer `alt` over `title`'
-  );
+  )
 
   t.equal(
     toString({
@@ -49,21 +42,15 @@ test('mdast-util-to-string', function (t) {
     }),
     'baz',
     'should use `title` over `children`'
-  );
+  )
 
   t.equal(
-    toString({
-      children: [{value: 'foo'}, {alt: 'bar'}, {title: 'baz'}]
-    }),
+    toString({children: [{value: 'foo'}, {alt: 'bar'}, {title: 'baz'}]}),
     'foobarbaz',
     'should prefer `children`'
-  );
+  )
 
-  t.equal(
-    toString({}),
-    '',
-    'should fall back on an empty string'
-  );
+  t.equal(toString({}), '', 'should fall back on an empty string')
 
-  t.end();
-});
+  t.end()
+})