Remove spurious reference to process.

Fixes #1257
graphql · Apr 18, 2018 · 60c03ab · 60c03ab
1 parent 69d90c6
commit 60c03ab
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/src/jsutils/instanceOf.js b/src/jsutils/instanceOf.js
@@ -16,8 +16,14 @@ declare function instanceOf(
   constructor: mixed,
 ): boolean %checks(value instanceof constructor);
 
-export default (process && process.env.NODE_ENV !== 'production'
+// See: https://expressjs.com/en/advanced/best-practice-performance.html#set-node_env-to-production
+// See: https://webpack.js.org/guides/production/
+export default (process.env.NODE_ENV === 'production'
   ? // eslint-disable-next-line no-shadow
+    function instanceOf(value: any, constructor: any) {
+      return value instanceof constructor;
+    }
+  : // eslint-disable-next-line no-shadow
     function instanceOf(value: any, constructor: any) {
       if (value instanceof constructor) {
         return true;
@@ -43,8 +49,4 @@ spurious results.`,
         }
       }
       return false;
-    }
-  : // eslint-disable-next-line no-shadow
-    function instanceOf(value: any, constructor: any) {
-      return value instanceof constructor;
     });