Refactor code-style

Author: wooorm

.prettierignore

@@ -0,0 +1,3 @@
+coverage/
+hast-util-menu-state.js
+hast-util-menu-state.min.js

index.js

@@ -1,42 +1,42 @@
-'use strict';
+'use strict'
 
-var is = require('hast-util-is-element');
+var is = require('hast-util-is-element')
 
-module.exports = menuState;
+module.exports = menuState
 
 /* Check the state of the menu `node`. */
 function menuState(nodes) {
-  var index;
-  var node;
-  var type;
+  var index
+  var node
+  var type
 
   if (!nodes || typeof nodes !== 'object' || nodes.length === 0) {
-    return null;
+    return null
   }
 
-  index = nodes.length - 1;
-  node = nodes[index];
+  index = nodes.length - 1
+  node = nodes[index]
 
   if (!is(node, 'menu')) {
-    return null;
+    return null
   }
 
   while (node) {
     /* Stop at `template` elements and non-elements. */
     if (!is(node) || is(node, 'template')) {
-      break;
+      break
     }
 
     if (is(node, 'menu')) {
-      type = node.properties && node.properties.type;
+      type = node.properties && node.properties.type
 
       if (type === 'context' || type === 'toolbar') {
-        return type;
+        return type
       }
     }
 
-    node = nodes[--index];
+    node = nodes[--index]
   }
 
-  return 'toolbar';
+  return 'toolbar'
 }

package.json

@@ -28,23 +28,31 @@
     "browserify": "^16.0.0",
     "esmangle": "^1.0.1",
     "nyc": "^12.0.0",
+    "prettier": "^1.13.5",
     "remark-cli": "^5.0.0",
     "remark-preset-wooorm": "^4.0.0",
     "tape": "^4.4.0",
     "xo": "^0.21.0"
   },
   "scripts": {
-    "build-md": "remark . --quiet --frail --output",
+    "format": "remark . -qfo && prettier --write '**/*.js' && xo --fix",
     "build-bundle": "browserify index.js --bare -s hastUtilMenuState > hast-util-menu-state.js",
     "build-mangle": "esmangle hast-util-menu-state.js > hast-util-menu-state.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,
     "ignores": [
       "hast-util-menu-state.js"

readme.md

@@ -13,31 +13,21 @@ npm install hast-util-menu-state
 ## Usage
 
 ```javascript
-var menuState = require('hast-util-menu-state');
+var menuState = require('hast-util-menu-state')
 
 // If there’s no last element or that node is not a `menu`:
-menuState([{type: 'element', tagName: 'a'}]); //=> null
+menuState([{type: 'element', tagName: 'a'}]) // => null
 
 // If the last node is a `menu` without `type`:
-menuState([{type: 'element', tagName: 'menu'}]); //=> 'toolbar'
+menuState([{type: 'element', tagName: 'menu'}]) // => 'toolbar'
 
 // If the last node is a `menu` with a `type`, or that node has
 // a parent `menu` with a type:
 menuState([
-  {
-    type: 'element',
-    tagName: 'menu',
-    properties: {type: 'context'}
-  },
-  {
-    type: 'element',
-    tagName: 'li'
-  },
-  {
-    type: 'element',
-    tagName: 'menu'
-  }
-]); //=> 'context'
+  {type: 'element', tagName: 'menu', properties: {type: 'context'}},
+  {type: 'element', tagName: 'li'},
+  {type: 'element', tagName: 'menu'}
+]) // => 'context'
 ```
 
 ## API

test.js

@@ -1,51 +1,59 @@
-'use strict';
+'use strict'
 
-var test = require('tape');
-var menuState = require('.');
+var test = require('tape')
+var menuState = require('.')
 
-test('menuState', function (t) {
-  t.equal(menuState(), null, 'should return `null` without nodes');
-  t.equal(menuState(null), null, 'should return `null` with `null`');
-  t.equal(menuState([]), null, 'should return `null` with empty `nodes`');
-  t.equal(menuState(['foo']), null, 'should return `null` without `node` as last `node`');
+test('menuState', function(t) {
+  t.equal(menuState(), null, 'should return `null` without nodes')
+  t.equal(menuState(null), null, 'should return `null` with `null`')
+  t.equal(menuState([]), null, 'should return `null` with empty `nodes`')
+  t.equal(
+    menuState(['foo']),
+    null,
+    'should return `null` without `node` as last `node`'
+  )
 
   t.equal(
     menuState([{type: 'text', value: 'alpha'}]),
     null,
     'should return `null` without `element` as last `node`'
-  );
+  )
 
   t.equal(
     menuState([{type: 'element', tagName: 'div'}]),
     null,
     'should return `null` without `menu` as last `node`'
-  );
+  )
 
   t.equal(
     menuState([{type: 'element', tagName: 'menu'}]),
     'toolbar',
     'should return `toolbar` without `type` on last `menu`'
-  );
+  )
 
   t.equal(
-    menuState([{
-      type: 'element',
-      tagName: 'menu',
-      properties: {type: 'toolbar'}
-    }]),
+    menuState([
+      {
+        type: 'element',
+        tagName: 'menu',
+        properties: {type: 'toolbar'}
+      }
+    ]),
     'toolbar',
     'should return `toolbar` with `type` set to `toolbar` on last `menu`'
-  );
+  )
 
   t.equal(
-    menuState([{
-      type: 'element',
-      tagName: 'menu',
-      properties: {type: 'context'}
-    }]),
+    menuState([
+      {
+        type: 'element',
+        tagName: 'menu',
+        properties: {type: 'context'}
+      }
+    ]),
     'context',
     'should return `context` with `type` set to `context` on last `menu`'
-  );
+  )
 
   t.equal(
     menuState([
@@ -58,7 +66,7 @@ test('menuState', function (t) {
     ]),
     'context',
     'should return the `type` of a parent `menu`, if available'
-  );
+  )
 
   t.equal(
     menuState([
@@ -67,7 +75,7 @@ test('menuState', function (t) {
     ]),
     'toolbar',
     'should return the `toolbar` of no parent `menu` is available'
-  );
+  )
 
   t.equal(
     menuState([
@@ -81,7 +89,7 @@ test('menuState', function (t) {
     ]),
     'toolbar',
     'should not walk higher than `template` elements'
-  );
+  )
 
   t.equal(
     menuState([
@@ -95,7 +103,7 @@ test('menuState', function (t) {
     ]),
     'toolbar',
     'should not walk higher than non-elements'
-  );
+  )
 
   t.equal(
     menuState([
@@ -109,7 +117,7 @@ test('menuState', function (t) {
     ]),
     'context',
     'should not walk higher than other elements'
-  );
+  )
 
-  t.end();
-});
+  t.end()
+})