fix: only access Node.js globals if available (#3501)

* fix: only access Node.js global if available * chore: adjust eslint config according to experts
graphql · May 9, 2022 · 97c9512 · 97c9512
1 parent a26f7c3
commit 97c9512
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/.eslintrc.yml b/.eslintrc.yml
@@ -743,3 +743,11 @@ overrides:
       # Ignore docusarus related webpack aliases
       import/no-unresolved:
         ['error', { 'ignore': ['^@theme', '^@docusaurus', '^@generated'] }]
+  - files:
+      - website/docusaurus.config.js
+      - website/sidebars.js
+      - integrationTests/**/*
+      - benchmark/**/*
+      - resources/**/*
+    env:
+      node: true
diff --git a/src/jsutils/instanceOf.ts b/src/jsutils/instanceOf.ts
@@ -9,8 +9,7 @@ import { inspect } from './inspect';
 export const instanceOf: (value: unknown, constructor: Constructor) => boolean =
   /* c8 ignore next 6 */
   // FIXME: https://github.com/graphql/graphql-js/issues/2317
-  // eslint-disable-next-line no-undef
-  process.env.NODE_ENV === 'production'
+  globalThis.process?.env.NODE_ENV === 'production'
     ? function instanceOf(value: unknown, constructor: Constructor): boolean {
         return value instanceof constructor;
       }