Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(transition): memory leaks #12190

Closed

Conversation

ferferga
Copy link
Contributor

Checks for memory leaks in the Transition component. These tests fails on master but succeeds on top of #12182

@edison1105 edison1105 added 🛑 on hold This PR can't be merged until other work is finished ready to merge The PR is ready to be merged. labels Oct 17, 2024
@edison1105 edison1105 deleted the branch vuejs:edison/fix/12181 October 21, 2024 07:36
@edison1105 edison1105 closed this Oct 21, 2024
@ferferga
Copy link
Contributor Author

@edison1105 Why were both PRs closed?

@edison1105 edison1105 reopened this Oct 21, 2024
@yyx990803 yyx990803 deleted the branch vuejs:edison/fix/12181 November 15, 2024 02:40
@yyx990803 yyx990803 closed this Nov 15, 2024
yyx990803 added a commit that referenced this pull request Nov 15, 2024
@yyx990803
Copy link
Member

Sorry, didn't realize this PR was pointing at #12181 when I merged it. Manually landed in 54812ea

noootwo pushed a commit to noootwo/core that referenced this pull request Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛑 on hold This PR can't be merged until other work is finished ready to merge The PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants