Refactor code-style

Author: wooorm

.prettierignore

@@ -0,0 +1,3 @@
+coverage/
+mdast-util-heading-style.js
+mdast-util-heading-style.min.js

index.js

@@ -1,17 +1,15 @@
-'use strict';
+'use strict'
 
-/* Expose. */
-module.exports = style;
+module.exports = style
 
-/* Check the style of a heading. */
 function style(node, relative) {
-  var last = node.children[node.children.length - 1];
-  var depth = node.depth;
-  var pos = node && node.position && node.position.end;
-  var final = last && last.position && last.position.end;
+  var last = node.children[node.children.length - 1]
+  var depth = node.depth
+  var pos = node && node.position && node.position.end
+  var final = last && last.position && last.position.end
 
   if (!pos) {
-    return null;
+    return null
   }
 
   /* This can only occur for `'atx'` and `'atx-closed'`
@@ -20,25 +18,29 @@ function style(node, relative) {
    * `'atx-closed'` heading. */
   if (!last) {
     if (pos.column - 1 <= depth * 2) {
-      return consolidate(depth, relative);
+      return consolidate(depth, relative)
     }
 
-    return 'atx-closed';
+    return 'atx-closed'
   }
 
   if (final.line + 1 === pos.line) {
-    return 'setext';
+    return 'setext'
   }
 
   if (final.column + depth < pos.column) {
-    return 'atx-closed';
+    return 'atx-closed'
   }
 
-  return consolidate(depth, relative);
+  return consolidate(depth, relative)
 }
 
 /* Get the probable style of an atx-heading, depending on
  * preferred style. */
 function consolidate(depth, relative) {
-  return depth < 3 ? 'atx' : relative === 'atx' || relative === 'setext' ? relative : null;
+  return depth < 3
+    ? 'atx'
+    : relative === 'atx' || relative === 'setext'
+      ? relative
+      : null
 }

package.json

@@ -25,6 +25,7 @@
   "devDependencies": {
     "browserify": "^16.0.0",
     "nyc": "^12.0.0",
+    "prettier": "^1.14.2",
     "remark": "^9.0.0",
     "remark-cli": "^5.0.0",
     "remark-preset-wooorm": "^4.0.0",
@@ -33,17 +34,24 @@
     "xo": "^0.22.0"
   },
   "scripts": {
-    "build-md": "remark . --quiet --frail --output",
+    "format": "remark . -qfo && prettier --write '**/*.js' && xo --fix",
     "build-bundle": "browserify . -s mdastUtilHeadingStyle > mdast-util-heading-style.js",
     "build-mangle": "browserify . -s mdastUtilHeadingStyle -p tinyify > mdast-util-heading-style.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-heading-style.js"

readme.md

@@ -13,15 +13,15 @@ npm install mdast-util-heading-style
 ## Usage
 
 ```js
-var style = require('mdast-util-heading-style');
-var remark = require('remark')();
+var style = require('mdast-util-heading-style')
+var remark = require('remark')()
 
-style(remark.parse('# ATX').children[0]); // 'atx'
-style(remark.parse('# ATX #\n').children[0]); // 'atx-closed'
-style(remark.parse('ATX\n===').children[0]); // 'setext'
+style(remark.parse('# ATX').children[0]) // => 'atx'
+style(remark.parse('# ATX #\n').children[0]) // => 'atx-closed'
+style(remark.parse('ATX\n===').children[0]) // => 'setext'
 
-style(remark.parse('### ATX').children[0]); // null
-style(remark.parse('### ATX').children[0], 'setext'); // 'setext'
+style(remark.parse('### ATX').children[0]) // => null
+style(remark.parse('### ATX').children[0], 'setext') // => 'setext'
 ```
 
 ## API

test.js

@@ -1,117 +1,108 @@
-'use strict';
+'use strict'
 
-var test = require('tape');
-var remark = require('remark')();
-var style = require('.');
+var test = require('tape')
+var remark = require('remark')()
+var style = require('.')
 
-test('mdast-util-heading-style', function (t) {
-  t.throws(
-    function () {
-      style();
-    },
-    'should fail without node'
-  );
+test('mdast-util-heading-style', function(t) {
+  t.throws(function() {
+    style()
+  }, 'should fail without node')
 
   t.equal(
     style({
       type: 'heading',
-      children: [
-        {type: 'text', value: 'foo'}
-      ]
+      children: [{type: 'text', value: 'foo'}]
     }),
     null,
     'should NOT fail on undetectable nodes'
-  );
+  )
 
-  t.equal(
-    style(remark.parse('# ATX').children[0]),
-    'atx',
-    'should detect atx'
-  );
+  t.equal(style(remark.parse('# ATX').children[0]), 'atx', 'should detect atx')
 
   t.equal(
     style(remark.parse('# ATX #').children[0]),
     'atx-closed',
     'should detect closed atx'
-  );
+  )
 
   t.equal(
     style(remark.parse('ATX\n===').children[0]),
     'setext',
     'should detect closed setext'
-  );
+  )
 
   t.equal(
     style(remark.parse('### ATX').children[0]),
     null,
     'should work on ambiguous nodes'
-  );
+  )
 
   t.equal(
     style(remark.parse('### ATX').children[0], 'atx'),
     'atx',
     'should work on ambiguous nodes (preference to atx)'
-  );
+  )
 
   t.equal(
     style(remark.parse('### ATX').children[0], 'setext'),
     'setext',
     'should work on ambiguous nodes (preference to setext)'
-  );
+  )
 
   t.equal(
     style(remark.parse('###### ######').children[0]),
     'atx-closed',
     'should work on empty nodes (#1)'
-  );
+  )
 
   t.equal(
     style(remark.parse('### ###').children[0]),
     'atx-closed',
     'should work on empty nodes (#2)'
-  );
+  )
 
   t.equal(
     style(remark.parse('# #').children[0]),
     'atx-closed',
     'should work on empty nodes (#3)'
-  );
+  )
 
   t.equal(
     style(remark.parse('###### ').children[0], 'atx'),
     'atx',
     'should work on empty nodes (#4)'
-  );
+  )
 
   t.equal(
     style(remark.parse('### ').children[0], 'atx'),
     'atx',
     'should work on empty nodes (#5)'
-  );
+  )
 
   t.equal(
     style(remark.parse('## ').children[0]),
     'atx',
     'should work on empty nodes (#6)'
-  );
+  )
 
   t.equal(
     style(remark.parse('###### ').children[0], 'setext'),
     'setext',
     'should work on empty nodes (#7)'
-  );
+  )
 
   t.equal(
     style(remark.parse('### ').children[0], 'setext'),
     'setext',
     'should work on empty nodes (#8)'
-  );
+  )
 
   t.equal(
     style(remark.parse('## ').children[0], 'setext'),
     'atx',
     'should work on empty nodes (#9)'
-  );
+  )
 
-  t.end();
-});
+  t.end()
+})