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

[BUGFIX lts] Update glimmer-vm to fixes knock on errors during cleanup. #18879

Merged
merged 1 commit into from
Apr 23, 2020

Conversation

rwjblue
Copy link
Member

@rwjblue rwjblue commented Apr 16, 2020

Backport of the changes in glimmerjs/glimmer-vm#1073 to 0.38.5-alpha branch (used by ember-source@3.16).

Fixes emberjs/ember-test-helpers#768 for ember-source@3.16.

Backport of the changes in
[glimmerjs/glimmer-vm#1073](glimmerjs/glimmer-vm#1073)
to 0.38.5-alpha branch (used by ember-source@3.16).
@rwjblue rwjblue added the Bug label Apr 16, 2020
@pzuraq
Copy link
Contributor

pzuraq commented Apr 16, 2020

Did we not have the same failing test on LTS as we did on master for this issue?

@rwjblue
Copy link
Member Author

rwjblue commented Apr 16, 2020

Did we not have the same failing test on LTS as we did on master for this issue?

No, that test only exists in beta branch (they landed only on master in #18782 roughly a month ago).

Copy link
Contributor

@pzuraq pzuraq left a comment

Choose a reason for hiding this comment

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

Ok, I think the new tests in the VM are probably enough

@rwjblue rwjblue merged commit 684941c into lts-3-16 Apr 23, 2020
@rwjblue rwjblue deleted the backport-prevent-knock-on-errors branch April 23, 2020 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants