Refactor code-style

Author: wooorm

.editorconfig

@@ -2,14 +2,8 @@ root = true
 
 [*]
 indent_style = space
-indent_size = 4
+indent_size = 2
 end_of_line = lf
 charset = utf-8
 trim_trailing_whitespace = true
 insert_final_newline = true
-
-[*.{json,html,svg,css,remarkrc,eslintrc}]
-indent_size = 2
-
-[*.md]
-trim_trailing_whitespace = false

index.js

@@ -1,44 +1,22 @@
-/**
- * @author Titus Wormer
- * @copyright 2015 Titus Wormer
- * @license MIT
- * @module mdast:util:to-string
- * @fileoverview Utility to get the plain text content of a node.
- */
-
 'use strict';
 
-/* eslint-env node */
-
-/**
- * Get the value of `node`.  Checks, `value`,
- * `alt`, and `title`, in that order.
- *
- * @param {Node} node - Node to get the internal value of.
- * @return {string} - Textual representation.
- */
-function valueOf(node) {
-    return node &&
-        (node.value ? node.value :
-        (node.alt ? node.alt : node.title)) || '';
-}
+/* Expose. */
+module.exports = toString;
 
-/**
- * Returns the text content of a node.  If the node itself
+/* Get the text content of a node.  If the node itself
  * does not expose plain-text fields, `toString` will
- * recursivly try its children.
- *
- * @param {Node} node - Node to transform to a string.
- * @return {string} - Textual representation.
- */
+ * recursivly try its children. */
 function toString(node) {
-    return valueOf(node) ||
-        (node.children && node.children.map(toString).join('')) ||
-        '';
+  return valueOf(node) ||
+    (node.children && node.children.map(toString).join('')) ||
+    '';
 }
 
-/*
- * Expose.
- */
-
-module.exports = toString;
+/* 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) || '';
+}

package.json

@@ -27,29 +27,31 @@
   "dependencies": {},
   "devDependencies": {
     "browserify": "^13.0.0",
-    "eslint": "^2.0.0",
     "esmangle": "^1.0.0",
     "istanbul": "^0.4.0",
-    "jscs": "^3.0.0",
-    "jscs-jsdoc": "^2.0.0",
     "remark-cli": "^1.0.0",
     "remark-comment-config": "^4.0.0",
     "remark-github": "^5.0.0",
     "remark-lint": "^4.0.0",
     "remark-slug": "^4.0.0",
     "remark-validate-links": "^4.0.0",
-    "tape": "^4.4.0"
+    "tape": "^4.4.0",
+    "xo": "^0.17.1"
   },
   "scripts": {
     "build-md": "remark . --quiet --frail",
     "build-bundle": "browserify index.js --no-builtins -s mdastUtilToString > mdast-util-to-string.js",
     "build-mangle": "esmangle mdast-util-to-string.js > mdast-util-to-string.min.js",
     "build": "npm run build-md && npm run build-bundle && npm run build-mangle",
-    "lint-api": "eslint .",
-    "lint-style": "jscs --reporter inline .",
-    "lint": "npm run lint-api && npm run lint-style",
+    "lint": "xo",
     "test-api": "node test.js",
     "test-coverage": "istanbul cover test.js",
     "test": "npm run build && npm run lint && npm run test-coverage"
+  },
+  "xo": {
+    "space": true,
+    "ignore": [
+      "mdast-util-to-string.js"
+    ]
   }
 }

readme.md

@@ -17,24 +17,13 @@ globals module, [uncompressed and compressed][releases].
 
 ## Usage
 
-Dependencies:
-
 ```js
 var remark = require('remark');
 var toString = require('mdast-util-to-string');
-```
 
-Parse:
+var tree = remark().parse('Some *emphasis*, **importance**, and `code`.');
 
-```js
-var ast = remark().parse('Some *emphasis*, **strongness**, and `code`.');
-```
-
-Stringify:
-
-```js
-toString(ast);
-// 'Some emphasis, strongness, and code.'
+console.log(toString(tree)); //=> 'Some emphasis, importance, and code.'
 ```
 
 ## API

test.js

@@ -1,94 +1,69 @@
-/**
- * @author Titus Wormer
- * @copyright 2015 Titus Wormer
- * @license MIT
- * @module mdast:util:to-string
- * @fileoverview Test suite for `mdast-util-to-string`.
- */
-
 'use strict';
 
-/* eslint-env node */
-
-/*
- * Dependencies.
- */
-
 var test = require('tape');
 var toString = require('./index.js');
 
-/*
- * Tests.
- */
-
 test('mdast-util-to-string', function (t) {
-    t.throws(
-        function () {
-            toString();
-        },
-        'should fail without node'
-    );
+  t.throws(
+    function () {
+      toString();
+    },
+    'should fail without node'
+  );
 
-    t.equal('foo', toString({
-        'value': 'foo'
-    }, 'should not fail on unrecognised nodes'));
+  t.equal(
+    toString({value: 'foo'}),
+    'foo',
+    'should not fail on unrecognised nodes'
+  );
 
-    t.equal('foo', toString({
-        'value': 'foo',
-        'children': [
-            {
-                'value': 'foo'
-            },
-            {
-                'alt': 'bar'
-            },
-            {
-                'title': 'baz'
-            }
-        ]
-    }), 'should prefer `value` over all others');
+  t.equal(
+    toString({
+      value: 'foo',
+      children: [{value: 'foo'}, {alt: 'bar'}, {title: 'baz'}]
+    }),
+    'foo',
+    'should prefer `value` over all others'
+  );
 
-    t.equal('foo', toString({
-        'value': 'foo',
-        'alt': 'bar',
-        'title': 'baz'
-    }), 'should prefer `value` over `alt` or `title`');
+  t.equal(
+    toString({
+      value: 'foo',
+      alt: 'bar',
+      title: 'baz'
+    }),
+    'foo',
+    'should prefer `value` over `alt` or `title`'
+  );
 
-    t.equal('bar', toString({
-        'alt': 'bar',
-        'title': 'baz'
-    }), 'should prefer `alt` over `title`');
+  t.equal(
+    toString({alt: 'bar', title: 'baz'}),
+    'bar',
+    'should prefer `alt` over `title`'
+  );
 
-    t.equal('baz', toString({
-        'title': 'baz',
-        'children': [
-            {
-                'value': 'foo'
-            },
-            {
-                'alt': 'bar'
-            },
-            {
-                'title': 'baz'
-            }
-        ]
-    }), 'should use `title` over `children`');
+  t.equal(
+    toString({
+      title: 'baz',
+      children: [{value: 'foo'}, {alt: 'bar'}, {title: 'baz'}]
+    }),
+    'baz',
+    'should use `title` over `children`'
+  );
 
-    t.equal('foobarbaz', toString({
-        'children': [
-            {
-                'value': 'foo'
-            },
-            {
-                'alt': 'bar'
-            },
-            {
-                'title': 'baz'
-            }
-        ]
-    }), 'should prefer `children`');
+  t.equal(
+    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();
 });