Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove core-js/regenerator-runtime stubs #6755

Merged
merged 1 commit into from
Nov 7, 2017
Merged

Remove core-js/regenerator-runtime stubs #6755

merged 1 commit into from
Nov 7, 2017

Conversation

existentialism
Copy link
Member

Q                       A
Fixed Issues?
Patch: Bug Fix? n
Major: Breaking Change? n
Minor: New Feature? n
Tests Added + Pass? n/y
Documentation PR
Any Dependency Changes?
License MIT

Removes the stubs originally added to eliminate confusion, but I think after much discussion, it wound up being more trouble than it was worth.

@existentialism existentialism added pkg: polyfill pkg: preset-env PR: Internal 🏠 A type of pull request used for our changelog categories labels Nov 6, 2017
@babel-bot
Copy link
Collaborator

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/5679/

@hzoo
Copy link
Member

hzoo commented Nov 6, 2017

Context: I believe I originally introduced these because of npm 2 but that is for node 4 and if that's going to be rarer we should just do this, maybe we can provide a better error message but having to maintain a list + this weird module redirection is just weird.

@hzoo hzoo requested review from zloirock and benjamn November 6, 2017 16:58
Copy link
Member

@zloirock zloirock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@hzoo hzoo merged commit ef47d8b into master Nov 7, 2017
@hzoo
Copy link
Member

hzoo commented Nov 7, 2017

Oh we need to change the readme if you didn't already, not sure if that can be a good first issue or not

@Andarist Andarist deleted the preset-env-core-js branch November 7, 2017 14:22
@existentialism
Copy link
Member Author

existentialism commented Nov 7, 2017

@hzoo I went ahead and made the updates: c440f04

futagoza added a commit to futagoza/babel-preset-futagozaryuu that referenced this pull request Nov 15, 2017
Added a script to allow dependencies from `@babel/polyfill` to work on Node.js v4 (see babel/babel#6755)
futagoza added a commit to futagoza/babel-preset-futagozaryuu that referenced this pull request Dec 12, 2017
While trying to find a way to let me use `@babel/polyfill: ^7.0.0-beta.32` on Node.js v4 I realised that I would have to do this with every package I create that supports Node.js v4, so I gave up.

Thanks babel/babel#6755
@lock lock bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Oct 5, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated A closed issue/PR that is archived due to age. Recommended to make a new issue pkg: polyfill pkg: preset-env PR: Internal 🏠 A type of pull request used for our changelog categories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants