Skip to content

[Babel 8] Remove extra.shorthand#16521

Merged
nicolo-ribaudo merged 2 commits into
babel:mainfrom
liuxingbaoyu:remove-extra-shorthand
Jun 3, 2024
Merged

[Babel 8] Remove extra.shorthand#16521
nicolo-ribaudo merged 2 commits into
babel:mainfrom
liuxingbaoyu:remove-extra-shorthand

Conversation

@liuxingbaoyu

@liuxingbaoyu liuxingbaoyu commented May 21, 2024

Copy link
Copy Markdown
Member
Q                       A
Fixed Issues? Fixes #1, Fixes #2
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link babel/website#2891
Any Dependency Changes?
License MIT

I changed testing tools to avoid a lot of duplication of tests.

@liuxingbaoyu liuxingbaoyu added the PR: Internal (next major) 🏠 A type of pull request used for our changelog categories for next major release label May 21, 2024
@babel-bot

Copy link
Copy Markdown
Collaborator

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

@JLHwung JLHwung left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

External tools might depend on the .extra.shorthand property, can we mark this PR as a breaking change? We should also update the migration docs for this change.

@nicolo-ribaudo nicolo-ribaudo added PR: Breaking Change 💥 A type of pull request used for our changelog categories for next major release PR: Needs Docs and removed PR: Internal (next major) 🏠 A type of pull request used for our changelog categories for next major release labels May 30, 2024
@nicolo-ribaudo

Copy link
Copy Markdown
Member

The recommended migration should be checking node.shorthand rather than node.extra.shorthand.

@nicolo-ribaudo
nicolo-ribaudo merged commit 17a5502 into babel:main Jun 3, 2024
@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 Sep 3, 2024
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 3, 2024
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 PR: Breaking Change 💥 A type of pull request used for our changelog categories for next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants