Skip to content

fix: skip global.js transform#17

Merged
FredKSchott merged 1 commit into
FredKSchott:mainfrom
underfin:skip-global-transform
Dec 3, 2021
Merged

fix: skip global.js transform#17
FredKSchott merged 1 commit into
FredKSchott:mainfrom
underfin:skip-global-transform

Conversation

@underfin

@underfin underfin commented Apr 29, 2021

Copy link
Copy Markdown
Contributor

Description

Here is code of global ployfill, the code has global key words, it will transformed and add import '\0polyfill-node:global.js'. But it is a self import, so we need skip it.

export default (typeof global !== "undefined" ? global :
  typeof self !== "undefined" ? self :
  typeof window !== "undefined" ? window : {});

@jacogr

jacogr commented Jul 8, 2021

Copy link
Copy Markdown

Indeed, using it results in the following -

(!) Circular dependencies
polyfill-node:global.js -> polyfill-node:global.js

A look/merge at this proposed solution would be appreciated.

@pkit

pkit commented Oct 1, 2021

Copy link
Copy Markdown

Aaand??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants