Skip to content

Conversation

JLHwung
Copy link
Contributor

@JLHwung JLHwung commented May 16, 2025

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

In this PR we imported unshiftForXStatementBody from the destructuring transform to handle the shadowed bindings within the for loop body.

@JLHwung JLHwung added PR: Bug Fix 🐛 A type of pull request used for our changelog categories Spec: Explicit Resource Management labels May 16, 2025
@babel-bot
Copy link
Collaborator

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

@nicolo-ribaudo nicolo-ribaudo merged commit 80c9a3a into babel:main May 23, 2025
57 checks passed
@nicolo-ribaudo nicolo-ribaudo deleted the fix-17318 branch May 23, 2025 09:41
@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 Aug 23, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 23, 2025
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: Bug Fix 🐛 A type of pull request used for our changelog categories Spec: Explicit Resource Management
Projects
None yet
Development

Successfully merging this pull request may close these issues.

explict-resource-management transform throws when for body binding shadows the for head using binding
4 participants