Refactor code-style

Author: wooorm

index.js

@@ -16,19 +16,16 @@ export const affixEmoticonModifier = modifyChildren(mergeAffixEmoticon)
  */
 function mergeAffixEmoticon(node, index, ancestor) {
   const previous = ancestor.children[index - 1]
-  var childIndex = -1
 
   if (index && parent(previous) && parent(node)) {
-    var children = node.children
+    const children = node.children
+    let childIndex = -1
 
     while (++childIndex < children.length) {
       const child = children[childIndex]
 
       if (child.type === 'EmoticonNode') {
-        previous.children = [].concat(
-          previous.children,
-          children.slice(0, childIndex + 1)
-        )
+        previous.children.push(...children.slice(0, childIndex + 1))
         node.children = children.slice(childIndex + 1)
 
         if (child.position && node.position && previous.position) {

package.json

@@ -70,12 +70,7 @@
     "trailingComma": "none"
   },
   "xo": {
-    "prettier": true,
-    "rules": {
-      "capitalized-comments": "off",
-      "no-var": "off",
-      "prefer-arrow-callback": "off"
-    }
+    "prettier": true
   },
   "remarkConfig": {
     "plugins": [

readme.md

@@ -33,7 +33,7 @@ import {emoticonModifier} from 'nlcst-emoticon-modifier'
 import {inspect} from 'unist-util-inspect'
 import {ParseEnglish} from 'parse-english'
 
-var english = new ParseEnglish()
+const english = new ParseEnglish()
 
 english.useFirst('tokenizeSentence', emoticonModifier)
 english.useFirst('tokenizeParagraph', affixEmoticonModifier)

test/index.js

@@ -14,18 +14,18 @@ import {removePosition} from 'unist-util-remove-position'
 import {affixEmoticonModifier} from '../index.js'
 
 /** @type {Node} */
-var lollipop = JSON.parse(
+const lollipop = JSON.parse(
   String(fs.readFileSync(path.join('test', 'fixtures', 'lollipop.json')))
 )
 
 /** @type {Node} */
-var smile = JSON.parse(
+const smile = JSON.parse(
   String(fs.readFileSync(path.join('test', 'fixtures', 'smile.json')))
 )
 
-test('affixEmoticonModifier()', function (t) {
+test('affixEmoticonModifier()', (t) => {
   t.throws(
-    function () {
+    () => {
       // @ts-ignore runtime.
       affixEmoticonModifier({})
     },
@@ -67,9 +67,10 @@ test('affixEmoticonModifier()', function (t) {
  * @param {boolean} [positionless=false]
  */
 function process(fixture, positionless) {
-  var processor = unified().use(english).use(plugin).freeze()
+  const processor = unified().use(english).use(plugin).freeze()
 
   if (positionless) {
+    // Fine.
     // type-coverage:ignore-next-line
     processor.Parser.prototype.position = false
   }
@@ -79,10 +80,13 @@ function process(fixture, positionless) {
 
 // Add modifier to processor.
 function plugin() {
+  // Fine.
   // type-coverage:ignore-next-line
   this.Parser.prototype.useFirst('tokenizeSentence', emojiModifier)
+  // Fine.
   // type-coverage:ignore-next-line
   this.Parser.prototype.useFirst('tokenizeSentence', emoticonModifier)
+  // Fine.
   // type-coverage:ignore-next-line
   this.Parser.prototype.useFirst('tokenizeParagraph', affixEmoticonModifier)
 }