Skip to content

polish: recommend preset for js extensions#11421

Merged
nicolo-ribaudo merged 1 commit intobabel:masterfrom
JLHwung:polish-missing-plugin-helpers
Apr 16, 2020
Merged

polish: recommend preset for js extensions#11421
nicolo-ribaudo merged 1 commit intobabel:masterfrom
JLHwung:polish-missing-plugin-helpers

Conversation

@JLHwung
Copy link
Copy Markdown
Contributor

@JLHwung JLHwung commented Apr 14, 2020

Q                       A
Fixed Issues? Per #11417 (review)
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

This PR recommends @babel/preset- whenver available. Besides that, it also mentions the syntax plugin. People may intend to leave it untranspiled, i.e. in a babel/syntax-typescript + tsc workflow.

@JLHwung JLHwung added PR: Polish 💅 A type of pull request used for our changelog categories area: errors pkg: core labels Apr 14, 2020
Copy link
Copy Markdown
Member

@hzoo hzoo left a comment

Choose a reason for hiding this comment

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

cool thanks 😄

@JLHwung JLHwung force-pushed the polish-missing-plugin-helpers branch from 3e6af4b to 7874fa2 Compare April 15, 2020 20:02
@nicolo-ribaudo
Copy link
Copy Markdown
Member

Thanks!

@nicolo-ribaudo nicolo-ribaudo merged commit 1c09701 into babel:master Apr 16, 2020
@nicolo-ribaudo nicolo-ribaudo deleted the polish-missing-plugin-helpers branch April 16, 2020 19:14
@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 Jul 17, 2020
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jul 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area: errors outdated A closed issue/PR that is archived due to age. Recommended to make a new issue pkg: core PR: Polish 💅 A type of pull request used for our changelog categories

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants