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,98 +1,65 @@
-/**
- * @author Titus Wormer
- * @copyright 2016 Titus Wormer
- * @license MIT
- * @module hast:util:parse-selector
- * @fileoverview Parse a simple CSS selector to a HAST node.
- */
-
 'use strict';
 
-/* eslint-env commonjs */
-
-/*
- * Characters
- */
+/* Expose. */
+module.exports = parse;
 
-var CC_DOT = '.'.charCodeAt(0);
-var CC_HASH = '#'.charCodeAt(0);
+/* Characters */
+var dot = '.'.charCodeAt(0);
+var hash = '#'.charCodeAt(0);
 
-/**
- * Parse a simple CSS selector into a HAST node.
- *
- * @example
- *   parse('foo#bar.baz.qux');
- *   // {
- *   //   'type': 'element',
- *   //   'tagName': 'foo',
- *   //   'properties': {
- *   //     'id': 'bar',
- *   //     'className': ['baz', 'qux']
- *   //   },
- *   //   'children': []
- *   // }
- *
- * @param {string?} selector - Simple CSS selector.
- * @return {Node} - HAST node.
- */
+/* Parse a simple CSS selector into a HAST node. */
 function parse(selector) {
-    var id = null;
-    var className = [];
-    var value = selector || '';
-    var name = 'div';
-    var node;
-    var type = null;
-    var index = -1;
-    var code;
-    var length = value.length;
-    var subvalue;
-    var lastIndex;
-
-    node = {
-        'type': 'element',
-        'tagName': null,
-        'properties': {},
-        'children': []
-    };
-
-    type = null;
-
-    while (++index <= length) {
-        code = value.charCodeAt(index);
-
-        if (!code || code === CC_DOT || code === CC_HASH) {
-            subvalue = value.slice(lastIndex, index);
-
-            if (subvalue) {
-                if (type === CC_DOT) {
-                    className.push(subvalue);
-                } else if (type === CC_HASH) {
-                    id = subvalue;
-                } else {
-                    name = subvalue;
-                }
-            }
-
-            lastIndex = index + 1;
-            type = code;
+  var id = null;
+  var className = [];
+  var value = selector || '';
+  var name = 'div';
+  var node;
+  var type = null;
+  var index = -1;
+  var code;
+  var length = value.length;
+  var subvalue;
+  var lastIndex;
+
+  node = {
+    type: 'element',
+    tagName: null,
+    properties: {},
+    children: []
+  };
+
+  type = null;
+
+  while (++index <= length) {
+    code = value.charCodeAt(index);
+
+    if (!code || code === dot || code === hash) {
+      subvalue = value.slice(lastIndex, index);
+
+      if (subvalue) {
+        if (type === dot) {
+          className.push(subvalue);
+        } else if (type === hash) {
+          id = subvalue;
+        } else {
+          name = subvalue;
         }
-    }
+      }
 
-    node.tagName = name;
-
-    if (id) {
-        node.properties.id = id;
+      lastIndex = index + 1;
+      type = code;
     }
+  }
 
-    if (className.length) {
-        node.properties.className = className;
-    }
+  node.tagName = name;
 
-    return node;
-}
+  if (id) {
+    node.properties.id = id;
+  }
 
-/*
- * Expose.
- */
+  if (className.length !== 0) {
+    node.properties.className = className;
+  }
 
-module.exports = parse;
+  return node;
+}

package.json

@@ -26,12 +26,9 @@
   "dependencies": {},
   "devDependencies": {
     "browserify": "^13.0.0",
-    "eslint": "^2.0.0",
     "esmangle": "^1.0.0",
     "hast": "0.0.2",
     "istanbul": "^0.4.0",
-    "jscs": "^2.0.0",
-    "jscs-jsdoc": "^1.0.0",
     "remark": "^4.0.0",
     "remark-comment-config": "^3.0.0",
     "remark-github": "^4.0.1",
@@ -40,18 +37,23 @@
     "remark-usage": "^3.0.0",
     "remark-validate-links": "^3.0.0",
     "remark-yaml-config": "^3.0.0",
-    "tape": "^4.0.0"
+    "tape": "^4.0.0",
+    "xo": "^0.17.0"
   },
   "scripts": {
     "build-md": "remark . --quiet --frail",
     "build-bundle": "browserify index.js --bare -s hastUtilParseSelector > hast-util-parse-selector.js",
     "build-mangle": "esmangle hast-util-parse-selector.js > hast-util-parse-selector.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-parse-selector.js"
+    ]
   }
 }

readme.md

@@ -15,34 +15,19 @@ module, [uncompressed and compressed][releases].
 
 ## Usage
 
-Dependencies:
-
 ```javascript
 var parseSelector = require('hast-util-parse-selector');
-```
 
-Parse:
-
-```javascript
-var node = parseSelector('.quux#bar.baz.qux');
+console.log(parseSelector('.quux#bar.baz.qux'));
 ```
 
 Yields:
 
-```json
-{
-  "type": "element",
-  "tagName": "div",
-  "properties": {
-    "id": "bar",
-    "className": [
-      "quux",
-      "baz",
-      "qux"
-    ]
-  },
-  "children": []
-}
+```js
+{ type: 'element',
+  tagName: 'div',
+  properties: { id: 'bar', className: [ 'quux', 'baz', 'qux' ] },
+  children: [] }
 ```
 
 ## API