refactor: use Array.from instead of spread

import-js · Jul 16, 2019 · 8dff5c8 · 8dff5c8
1 parent c0bbcb6
commit 8dff5c8
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/rules/export.js b/src/rules/export.js
@@ -31,12 +31,12 @@ const tsTypePrefix = 'type:'
  * export function foo(a: string);
  * export function foo(a: number|string) { return a; }
  * ```
- * @param {Object[]} nodes
+ * @param {Set<Object>} nodes
  * @returns {boolean}
  */
 function isTypescriptFunctionOverloads(nodes) {
-  return nodes.some(node => node.parent.type === 'TSDeclareFunction') &&
-    nodes.every(node => (
+  return Array.from(nodes).some(node => node.parent.type === 'TSDeclareFunction') &&
+    Array.from(nodes).every(node => (
       node.parent.type === 'TSDeclareFunction' ||
       node.parent.type === 'FunctionDeclaration'
     ))
@@ -141,7 +141,7 @@ module.exports = {
           for (let [name, nodes] of named) {
             if (nodes.size <= 1) continue
 
-            if (isTypescriptFunctionOverloads([...nodes])) continue
+            if (isTypescriptFunctionOverloads(nodes)) continue
 
             for (let node of nodes) {
               if (name === 'default') {