tools,lib: remove global and Intl from the list of globals

Refs: #38230 Refs: #38419 PR-URL: #42014 Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Shingo Inoue <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rich Trott <[email protected]>
nodejs · Apr 24, 2022 · 80f944f · 80f944f
1 parent 797bf62
commit 80f944f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml
@@ -171,9 +171,7 @@ rules:
     - name: WeakSet
       into: Safe
 globals:
-  Intl: false
   # Parameters passed to internal modules
-  global: false
   require: false
   process: false
   exports: false

diff --git a/lib/internal/per_context/primordials.js b/lib/internal/per_context/primordials.js
@@ -162,7 +162,7 @@ function copyPrototype(src, dest, prefix) {
   'Reflect',
 ].forEach((name) => {
   // eslint-disable-next-line no-restricted-globals
-  copyPropsRenamed(global[name], primordials, name);
+  copyPropsRenamed(globalThis[name], primordials, name);
 });
 
 // Create copies of intrinsic objects
@@ -206,7 +206,7 @@ function copyPrototype(src, dest, prefix) {
   'WeakSet',
 ].forEach((name) => {
   // eslint-disable-next-line no-restricted-globals
-  const original = global[name];
+  const original = globalThis[name];
   primordials[name] = original;
   copyPropsRenamed(original, primordials, name);
   copyPrototype(original.prototype, primordials, `${name}Prototype`);
@@ -219,7 +219,7 @@ function copyPrototype(src, dest, prefix) {
   'Promise',
 ].forEach((name) => {
   // eslint-disable-next-line no-restricted-globals
-  const original = global[name];
+  const original = globalThis[name];
   primordials[name] = original;
   copyPropsRenamedBound(original, primordials, name);
   copyPrototype(original.prototype, primordials, `${name}Prototype`);