Skip to content

[babel 8] Remove block argument from Scope#rename#15288

Merged
nicolo-ribaudo merged 1 commit into
babel:mainfrom
nicolo-ribaudo:babel-8-remove-rename-block-param
Jan 12, 2023
Merged

[babel 8] Remove block argument from Scope#rename#15288
nicolo-ribaudo merged 1 commit into
babel:mainfrom
nicolo-ribaudo:babel-8-remove-rename-block-param

Conversation

@nicolo-ribaudo
Copy link
Copy Markdown
Member

@nicolo-ribaudo nicolo-ribaudo commented Dec 16, 2022

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

I noticed in #15287 that this param isn't actually used.

@babel-bot
Copy link
Copy Markdown
Collaborator

babel-bot commented Dec 16, 2022

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

@nicolo-ribaudo nicolo-ribaudo force-pushed the babel-8-remove-rename-block-param branch from 05c1c39 to 8e481ce Compare January 12, 2023 22:20
@nicolo-ribaudo nicolo-ribaudo merged commit 7a4ca1b into babel:main Jan 12, 2023
@nicolo-ribaudo nicolo-ribaudo deleted the babel-8-remove-rename-block-param branch January 12, 2023 22:36
@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 Apr 14, 2023
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 14, 2023
@JLHwung JLHwung added PR: Breaking Change 💥 A type of pull request used for our changelog categories for next major release and removed babel 8 labels Aug 9, 2023
@JLHwung JLHwung added this to the v8.0.0 milestone Aug 9, 2023
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