Skip to content

Move environment-visitor helper into @babel/traverse#16649

Merged
nicolo-ribaudo merged 1 commit intofeat-7.25.0/remove-cyclesfrom
revert-16647-revert-16577-remove-cycles-1
Jul 17, 2024
Merged

Move environment-visitor helper into @babel/traverse#16649
nicolo-ribaudo merged 1 commit intofeat-7.25.0/remove-cyclesfrom
revert-16647-revert-16577-remove-cycles-1

Conversation

@nicolo-ribaudo
Copy link
Member

This was landed in #16577 and reverted in #16647 because merged too soon. It has already been reviewed.

@nicolo-ribaudo nicolo-ribaudo added PR: New Feature 🚀 A type of pull request used for our changelog categories pkg: traverse PR: Ready to be Merged A pull request with already two approvals, but waiting for the next minor release labels Jul 16, 2024
@nicolo-ribaudo nicolo-ribaudo added this to the v7.25.0 milestone Jul 16, 2024
@babel-bot
Copy link
Collaborator

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

@nicolo-ribaudo nicolo-ribaudo changed the base branch from main to feat-7.25.0/remove-cycles July 17, 2024 12:02
@nicolo-ribaudo nicolo-ribaudo merged commit 406ba06 into feat-7.25.0/remove-cycles Jul 17, 2024
@nicolo-ribaudo nicolo-ribaudo deleted the revert-16647-revert-16577-remove-cycles-1 branch July 17, 2024 12:03
ArrowFunctionExpression(path) {
path.skip();
},
const awaitVisitor = visitors.environmentVisitor<{ wrapAwait: t.Expression }>({

Choose a reason for hiding this comment

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

_traverse.visitors.environmentVisitor is not a function

Copy link

@AustinGitHub AustinGitHub Jul 30, 2024

Choose a reason for hiding this comment

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

Hi, Yes, this PR cause an eslint issue in our codebase repo, this is the version we are on.

Screenshot 2024-07-30 at 8 59 49 AM

Not sure if maybe extensions we use are on latest babels, I have eslint extension installed on vs code as well.
Is there an issue opened or plans to fix this? Thanks!

@nicolo-ribaudo

@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 Oct 30, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 30, 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 pkg: traverse 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.

4 participants