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

Allow defining the moduleIds-related option in the transform plugins #11194

Conversation

nicolo-ribaudo
Copy link
Member

@nicolo-ribaudo nicolo-ribaudo commented Mar 1, 2020

Q                       A
Fixed Issues? Fixes #5473
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature? Yes
Tests Added + Pass? Yes
Documentation PR Link babel/website#2775
Any Dependency Changes?
License MIT

This PR is the first step towards #5473. I didn't move filenameRelative since it's not mentioned in the modules-related docs, and it's used as a default value for sourceFileName's option.

When this PR is merged, I will open one in the next-8-dev branch to remove the global options. This will make it easier to migrate before upgrading to Babel 8.

@nicolo-ribaudo nicolo-ribaudo added PR: New Feature 🚀 A type of pull request used for our changelog categories area: modules pkg: core labels Mar 1, 2020
@@ -46,6 +46,7 @@
"@babel/template": "^7.8.6",
"@babel/traverse": "^7.8.6",
"@babel/types": "^7.8.6",
"@babel/helper-module-transforms": "^7.8.6",
Copy link
Member

Choose a reason for hiding this comment

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

Nit: alpha sort

Copy link
Member

@existentialism existentialism left a comment

Choose a reason for hiding this comment

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

Minor nit, but otherwise LGTM!

@nicolo-ribaudo nicolo-ribaudo mentioned this pull request Mar 3, 2020
34 tasks
@nicolo-ribaudo nicolo-ribaudo added the PR: Ready to be Merged A pull request with already two approvals, but waiting for the next minor release label Mar 3, 2020
@nicolo-ribaudo nicolo-ribaudo added this to the v7.9.0 milestone Mar 3, 2020
@nicolo-ribaudo nicolo-ribaudo force-pushed the add-module-ids-opts-to-transform-plugins branch from 009e29a to 26800f5 Compare March 16, 2020 21:56
@nicolo-ribaudo nicolo-ribaudo merged commit a875560 into babel:master Mar 16, 2020
@nicolo-ribaudo nicolo-ribaudo deleted the add-module-ids-opts-to-transform-plugins branch March 16, 2020 22:58
@github-actions github-actions bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Jun 16, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area: modules outdated A closed issue/PR that is archived due to age. Recommended to make a new issue pkg: core PR: New Feature 🚀 A type of pull request used for our changelog categories PR: Ready to be Merged A pull request with already two approvals, but waiting for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[7.0] Consider moving root AMD/UMD/SystemJS options to be plugin/preset options?
3 participants