Skip to content
This repository was archived by the owner on Feb 1, 2025. It is now read-only.

Process polyfill included in builds #289

Closed
Andarist opened this issue Apr 25, 2017 · 2 comments · Fixed by #291
Closed

Process polyfill included in builds #289

Andarist opened this issue Apr 25, 2017 · 2 comments · Fixed by #291

Comments

@Andarist
Copy link
Contributor

I'm wondering if this could have been avoided.

More information can be found in related thread

@benjamn
Copy link
Contributor

benjamn commented Apr 26, 2017

Would this be enough to prevent the polyfill?

if (global.process && global.process.domain) {
  invoke = global.process.domain.bind(invoke);
}

Andarist added a commit to Andarist/regenerator that referenced this issue Apr 26, 2017
@Andarist
Copy link
Contributor Author

Just checked it and created PR with this guard included.

It still triggers global's polyfill, but this one is really small, so rather a non-issue. This is really weird webpack's implicit behaviour, this whole auto-polyfilling stuff, and cant find any information about this - would like to configure it in the configuration setup. Anyway - this simple 'fix' will let people ship a little bit less code to the browser.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants