@parcel/transformer-babel: remove core-js dep

[fregante]

↪️ Pull Request

core-js isn't a direct dependency of @parcel/transformer-babel so it should not be listed as one.

Closes #4433

✔️ PR Todo

• [/] Added/updated unit tests for this change
• [/] Filled out test instructions (In case there aren't any unit tests)
• Included links to related issues/PRs

[height]

Link Height tasks by mentioning a task ID in the pull request title or description, commit messages, or comments.

You can also use "Close T-X" to automatically close a task when the pull request is merged.

[fregante]

Cmd.exe exited with code '-1073741819'.

I think that windows test failed due to an unrelated memory issue. Maybe try re-running it.

[mischnic]

As I've already commented in #4433, @babel/preset-env uses core-js if useBuiltins is set, which we do here for the default babel config:

parcel/packages/transformers/babel/src/env.js

Lines 39 to 40 in f01437e

	useBuiltIns: 'entry',
	corejs: 3,

According to babel, you should install core-js in that case:

WARNING: We noticed you're using the useBuiltIns option without declaring a core-js version. Currently, we assume version 2.x when no version is passed. Since this default version will likely change in future versions of Babel, we recommend explicitly setting the core-js version you are using via the corejs option.

You should also be sure that the version you pass to the corejs option matches the version specified in your package.json's dependencies section. If it doesn't, you need to run one of the following commands:

npm install --save core-js@2 npm install --save core-js@3
yarn add core-js@2 yarn add core-js@3

https://github.com/babel/babel/blob/aeebc0823415ed4770c633cedf570cf774ae89c6/packages/babel-preset-env/src/normalize-options.js#L211-L222

[fregante]

According to babel, you should install core-js in that case:

Indeed, so #4433 is invalid and should be closed?