Refactor code-style

Author: wooorm

.prettierignore

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

index.js

@@ -1,51 +1,51 @@
-'use strict';
+'use strict'
 
-var visit = require('unist-util-visit');
+var visit = require('unist-util-visit')
 
-module.exports = getDefinitionFactory;
+module.exports = getDefinitionFactory
 
-var own = {}.hasOwnProperty;
+var own = {}.hasOwnProperty
 
 /* Get a definition in `node` by `identifier`. */
 function getDefinitionFactory(node, options) {
-  return getterFactory(gather(node, options));
+  return getterFactory(gather(node, options))
 }
 
 /* Gather all definitions in `node` */
 function gather(node, options) {
-  var cache = {};
+  var cache = {}
 
   if (!node || !node.type) {
-    throw new Error('mdast-util-definitions expected node');
+    throw new Error('mdast-util-definitions expected node')
   }
 
-  visit(node, 'definition', options && options.commonmark ? commonmark : normal);
+  visit(node, 'definition', options && options.commonmark ? commonmark : normal)
 
-  return cache;
+  return cache
 
   function commonmark(definition) {
-    var id = normalise(definition.identifier);
+    var id = normalise(definition.identifier)
     if (!own.call(cache, id)) {
-      cache[id] = definition;
+      cache[id] = definition
     }
   }
 
   function normal(definition) {
-    cache[normalise(definition.identifier)] = definition;
+    cache[normalise(definition.identifier)] = definition
   }
 }
 
 /* Factory to get a node from the given definition-cache. */
 function getterFactory(cache) {
-  return getter;
+  return getter
 
   /* Get a node from the bound definition-cache. */
   function getter(identifier) {
-    var id = identifier && normalise(identifier);
-    return id && own.call(cache, id) ? cache[id] : null;
+    var id = identifier && normalise(identifier)
+    return id && own.call(cache, id) ? cache[id] : null
   }
 }
 
 function normalise(identifier) {
-  return identifier.toUpperCase();
+  return identifier.toUpperCase()
 }

package.json

@@ -27,6 +27,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",
@@ -35,17 +36,24 @@
     "xo": "^0.22.0"
   },
   "scripts": {
-    "build-md": "remark . --quiet --frail --output",
+    "format": "remark . -qfo && prettier --write '**/*.js' && xo --fix",
     "build-bundle": "browserify . -s mdastUtilDefinitions > mdast-util-definitions.js",
     "build-mangle": "browserify . -s mdastUtilDefinitions -p tinyify > mdast-util-definitions.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-definitions.js"

readme.md

@@ -14,18 +14,18 @@ npm install mdast-util-definitions
 ## Usage
 
 ```js
-var remark = require('remark');
-var definitions = require('mdast-util-definitions');
+var remark = require('remark')
+var definitions = require('mdast-util-definitions')
 
-var ast = remark().parse('[example]: http://example.com "Example"');
+var ast = remark().parse('[example]: http://example.com "Example"')
 
-var definition = definitions(ast);
+var definition = definitions(ast)
 
-definition('example');
-// {type: 'definition', 'title': 'Example', ...}
+definition('example')
+// => {type: 'definition', 'title': 'Example', ...}
 
-definition('foo');
-// null
+definition('foo')
+// => null
 ```
 
 ## API

test.js

@@ -1,23 +1,23 @@
-'use strict';
+'use strict'
 
-var test = require('tape');
-var remark = require('remark');
-var definitions = require('.');
+var test = require('tape')
+var remark = require('remark')
+var definitions = require('.')
 
-test('mdast-util-definitions', function (t) {
-  var getDefinition;
-  var tree;
+test('mdast-util-definitions', function(t) {
+  var getDefinition
+  var tree
 
   t.throws(
-    function () {
-      definitions();
+    function() {
+      definitions()
     },
     /mdast-util-definitions expected node/,
     'should fail without node'
-  );
+  )
 
-  tree = remark().parse('[example]: http://example.com "Example"');
-  getDefinition = definitions(tree);
+  tree = remark().parse('[example]: http://example.com "Example"')
+  getDefinition = definitions(tree)
 
   t.deepEqual(
     getDefinition('example'),
@@ -33,16 +33,12 @@ test('mdast-util-definitions', function (t) {
       }
     },
     'should return a definition'
-  );
+  )
 
-  t.equal(
-    getDefinition('foo'),
-    null,
-    'should return null when not found'
-  );
+  t.equal(getDefinition('foo'), null, 'should return null when not found')
 
-  tree = remark().parse('[__proto__]: http://proto.com "Proto"');
-  getDefinition = definitions(tree);
+  tree = remark().parse('[__proto__]: http://proto.com "Proto"')
+  getDefinition = definitions(tree)
 
   t.deepEqual(
     getDefinition('__proto__'),
@@ -58,30 +54,27 @@ test('mdast-util-definitions', function (t) {
       }
     },
     'should work on weird identifiers'
-  );
+  )
 
   t.deepEqual(
     getDefinition('toString'),
     null,
     'should work on weird identifiers when not found'
-  );
+  )
 
-  tree = remark().parse([
-    '[example]: http://one.com',
-    '[example]: http://two.com'
-  ].join('\n'));
+  tree = remark().parse('[example]: http://one.com\n[example]: http://two.com')
 
   t.deepEqual(
     definitions(tree)('example').url,
     'http://two.com',
     'should prefer the last of duplicate definitions by default'
-  );
+  )
 
   t.deepEqual(
     definitions(tree, {commonmark: true})('example').url,
     'http://one.com',
     'should prefer the first of duplicate definitions in commonmark mode'
-  );
+  )
 
-  t.end();
-});
+  t.end()
+})