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,remarkrc,eslintrc,sh}]
-indent_size = 2
-
-[*.md]
-trim_trailing_whitespace = false

index.js

@@ -1,48 +1,23 @@
-/**
- * @author Titus Wormer
- * @copyright 2016 Titus Wormer
- * @license MIT
- * @module hast:util:whitespace
- * @fileoverview Check whether a node is inter-element
- *   whitespace.
- */
-
 'use strict';
 
-/* eslint-env commonjs */
-
-/*
- * HTML white-space expression.
- *
- * See <https://html.spec.whatwg.org/#space-character>.
- */
+/* Expose. */
+module.exports = interElementWhiteSpace;
 
-var EXPRESSION = /[\ \t\n\f\r]/g;
+/* HTML white-space expression.
+ * See <https://html.spec.whatwg.org/#space-character>. */
+var re = /[ \t\n\f\r]/g;
 
-/**
- * Check if `node` is a inter-element white-space.
- *
- * @param {Node|string} node - Value to check, or Node
- *   whose value to check.
- * @return {boolean} - Whether `node` is inter-element
- *   white-space.
- */
+/* Check if `node` is a inter-element white-space. */
 function interElementWhiteSpace(node) {
-    var value;
+  var value;
 
-    if (node && typeof node === 'object' && node.type === 'text') {
-        value = node.value || ''
-    } else if (typeof node === 'string') {
-        value = node;
-    } else {
-        return false;
-    }
+  if (node && typeof node === 'object' && node.type === 'text') {
+    value = node.value || '';
+  } else if (typeof node === 'string') {
+    value = node;
+  } else {
+    return false;
+  }
 
-    return value.replace(EXPRESSION, '') === '';
+  return value.replace(re, '') === '';
 }
-
-/*
- * Expose.
- */
-
-module.exports = interElementWhiteSpace;

package.json

@@ -27,29 +27,31 @@
   ],
   "devDependencies": {
     "browserify": "^13.0.0",
-    "eslint": "^2.0.0",
     "esmangle": "^1.0.1",
     "istanbul": "^0.4.0",
-    "jscs": "^3.0.0",
-    "jscs-jsdoc": "^2.0.0",
     "remark": "^4.0.0",
     "remark-comment-config": "^3.0.0",
     "remark-github": "^4.0.1",
     "remark-lint": "^3.0.0",
     "remark-usage": "^3.0.0",
     "remark-validate-links": "^3.0.0",
-    "tape": "^4.4.0"
+    "tape": "^4.4.0",
+    "xo": "^0.17.0"
   },
   "scripts": {
     "build-md": "remark . --quiet --frail",
     "build-bundle": "browserify index.js --bare -s hastUtilWhitespace > hast-util-whitespace.js",
     "build-mangle": "esmangle hast-util-whitespace.js > hast-util-whitespace.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,
+    "ignores": [
+      "hast-util-is-element.js"
+    ]
   }
 }

readme.md

@@ -20,32 +20,22 @@ Dependencies:
 
 ```javascript
 var whitespace = require('hast-util-whitespace');
-```
-
-Checks:
-
-```javascript
-var a = whitespace({
-    'type': 'element',
-    'tagName': 'div',
-    'children': []
-});
-var b = whitespace({
-    'type': 'text',
-    'value': '\t  \n'
-});
-var c = whitespace({
-    'type': 'text',
-    'value': '  text\f'
-});
-```
-
-Yields:
 
-```txt
-a: false
-b: true
-c: false
+whitespace({
+  type: 'element',
+  tagName: 'div',
+  children: []
+}); //=> false
+
+whitespace({
+  type: 'text',
+  value: '\t  \n'
+}); //=> true
+
+whitespace({
+  type: 'text',
+  value: '  text\f'
+}); //=> false
 ```
 
 ## API

test.js

@@ -1,59 +1,48 @@
-/**
- * @author Titus Wormer
- * @copyright 2016 Titus Wormer
- * @license MIT
- * @module hast:util:whitespace
- * @fileoverview Test suite for `whitespace`.
- */
-
 'use strict';
 
-/* eslint-env node */
-
-/*
- * Module dependencies.
- */
-
+/* Dependencies. */
 var test = require('tape');
 var whitespace = require('./index.js');
 
-/*
- * Tests.
- */
-
+/* Tests. */
 test('whitespace', function (t) {
-    t.equal(whitespace(), false, 'should return `false` without node');
-
-    t.equal(whitespace({
-        'type': 'element',
-        'tagName': 'div'
-    }), false, 'should return `false` without text');
-
-    t.equal(whitespace({
-        'type': 'text',
-        'value': '\v'
-    }), false, 'should return `false` for other white-space');
-
-    t.equal(whitespace({
-        'type': 'text',
-        'value': ' \t\r\n\f'
-    }), true, 'should return `true` for inter-element white-space');
-
-    t.equal(whitespace({
-        'type': 'text'
-    }), true, 'should return `true` for `text` without value');
-
-    t.equal(
-        whitespace(' \v'),
-        false,
-        'should return `false` for a `string` of text'
-    );
-
-    t.equal(
-        whitespace(' \t\r\n\f'),
-        true,
-        'should return `true` for a `string` of inter-element white-space'
-    );
-
-    t.end();
+  t.equal(whitespace(), false, 'should return `false` without node');
+
+  t.equal(
+    whitespace({type: 'element', tagName: 'div'}),
+    false,
+    'should return `false` without text'
+  );
+
+  t.equal(
+    whitespace({type: 'text', value: '\v'}),
+    false,
+    'should return `false` for other white-space'
+  );
+
+  t.equal(
+    whitespace({type: 'text', value: ' \t\r\n\f'}),
+    true,
+    'should return `true` for inter-element white-space'
+  );
+
+  t.equal(
+    whitespace({type: 'text'}),
+    true,
+    'should return `true` for `text` without value'
+  );
+
+  t.equal(
+    whitespace(' \v'),
+    false,
+    'should return `false` for a `string` of text'
+  );
+
+  t.equal(
+    whitespace(' \t\r\n\f'),
+    true,
+    'should return `true` for a `string` of inter-element white-space'
+  );
+
+  t.end();
 });