From 58832ae2468afbf377f9948ed79e09a86098ee2a Mon Sep 17 00:00:00 2001
From: Ivan Goncharov <ivan.goncharov.ua@gmail.com>
Date: Tue, 21 May 2019 16:13:41 +0300
Subject: [PATCH] Enable "no-sequences" ESLint rule (#1883)

---
 .eslintrc.yml            | 2 +-
 src/jsutils/keyMap.js    | 8 ++++----
 src/jsutils/keyValMap.js | 8 ++++----
 src/type/schema.js       | 8 ++++----
 4 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/.eslintrc.yml b/.eslintrc.yml
index d0c34761ac..572cf760e7 100644
--- a/.eslintrc.yml
+++ b/.eslintrc.yml
@@ -149,7 +149,7 @@ rules:
   no-script-url: error
   no-self-assign: error
   no-self-compare: off # TODO
-  no-sequences: off # TODO
+  no-sequences: error
   no-throw-literal: error
   no-unmodified-loop-condition: error
   no-unused-expressions: error
diff --git a/src/jsutils/keyMap.js b/src/jsutils/keyMap.js
index 26a7da9d9d..d181b2de3f 100644
--- a/src/jsutils/keyMap.js
+++ b/src/jsutils/keyMap.js
@@ -36,8 +36,8 @@ export default function keyMap<T>(
   list: $ReadOnlyArray<T>,
   keyFn: (item: T) => string,
 ): ObjMap<T> {
-  return list.reduce(
-    (map, item) => ((map[keyFn(item)] = item), map),
-    Object.create(null),
-  );
+  return list.reduce((map, item) => {
+    map[keyFn(item)] = item;
+    return map;
+  }, Object.create(null));
 }
diff --git a/src/jsutils/keyValMap.js b/src/jsutils/keyValMap.js
index f0823605c3..772b43b00b 100644
--- a/src/jsutils/keyValMap.js
+++ b/src/jsutils/keyValMap.js
@@ -31,8 +31,8 @@ export default function keyValMap<T, V>(
   keyFn: (item: T) => string,
   valFn: (item: T) => V,
 ): ObjMap<V> {
-  return list.reduce(
-    (map, item) => ((map[keyFn(item)] = valFn(item)), map),
-    Object.create(null),
-  );
+  return list.reduce((map, item) => {
+    map[keyFn(item)] = valFn(item);
+    return map;
+  }, Object.create(null));
 }
diff --git a/src/type/schema.js b/src/type/schema.js
index 5ea5164c1f..550298b010 100644
--- a/src/type/schema.js
+++ b/src/type/schema.js
@@ -257,10 +257,10 @@ export class GraphQLSchema {
 
     if (!possibleTypeMap[abstractType.name]) {
       const possibleTypes = this.getPossibleTypes(abstractType);
-      possibleTypeMap[abstractType.name] = possibleTypes.reduce(
-        (map, type) => ((map[type.name] = true), map),
-        Object.create(null),
-      );
+      possibleTypeMap[abstractType.name] = possibleTypes.reduce((map, type) => {
+        map[type.name] = true;
+        return map;
+      }, Object.create(null));
     }
 
     return Boolean(possibleTypeMap[abstractType.name][possibleType.name]);