graphql-js
fix: remove `globalThis` check and align with what bundlers can accept
#4022

Merged

fix: remove `globalThis` check and align with what bundlers can accept #4022

saihaj merged 8 commits into graphql:16.x.x from JoviDeCroock:fix-node-env-issue

remove globalThis process env check

546c13f4

benjie approved these changes on 2024-02-14

JoviDeCroock force pushed from 5aa81211 to daffdfff 1 year ago

JoviDeCroock force pushed from daffdfff to a8f5b4a5 1 year ago

JoviDeCroock force pushed from a8f5b4a5 to b0ca7e03 1 year ago

add eslint ignore as there was before

b0ca7e03

JoviDeCroock commented on 2024-05-01

n1ru4l approved these changes on 2024-05-02

JoviDeCroock commented on 2024-05-04

Update src/jsutils/instanceOf.ts

cbb05cf4

remove unused directive

0b8e1079

update check to use globalThis.process

157490b2

add docs

5b3f00e0

saihaj approved these changes on 2024-05-29

run prettier

6b6d7d3b

saihaj added PR: bug fix 🐞

add spelling checks

f57c710f

saihaj merged 0d12b065 into 16.x.x 1 year ago

Thompson1985 approved these changes on 2024-12-04

Reviewers

saihaj

benjie

Thompson1985

n1ru4l

echocrow

Assignees

No one assigned

Labels

PR: bug fix 🐞

Milestone

No milestone