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 beta] Adding tests for #18732 #18751

Conversation

GCheung55
Copy link
Contributor

[BUGFIX beta] Adding tests for #18732

@chancancode chancancode changed the base branch from bugfix/upgrade-glimmer-vm-0.47.9 to master February 20, 2020 21:54
@chancancode
Copy link
Member

Changing the base branch to master temporarily to confirm the tests do fail

@chancancode
Copy link
Member

Hm. I guess that doesn't work

@chancancode chancancode force-pushed the bugfix/upgrade-glimmer-vm-0.47.9 branch from 712da09 to 699439c Compare February 20, 2020 21:57
@chancancode chancancode force-pushed the bugfix/upgrade-glimmer-vm-0.47.9 branch from 699439c to 004cea1 Compare February 20, 2020 21:59
@chancancode
Copy link
Member

chancancode commented Feb 20, 2020

Ok I think I did it (sorry, had to force push to your branch). If it fails correctly, I think we can just change the base branch back to @pzuraq's branch. (I haven't looked at the actual tests being added yet.)

@GCheung55
Copy link
Contributor Author

@chancancode It failed, however, I'm unsure whether it failed expectedly.

@rwjblue rwjblue changed the base branch from master to bugfix/upgrade-glimmer-vm-0.47.9 February 20, 2020 23:21
@rwjblue
Copy link
Member

rwjblue commented Feb 20, 2020

Failure looks like I’d expect, updating target branch back.

@rwjblue rwjblue merged commit 76abb62 into emberjs:bugfix/upgrade-glimmer-vm-0.47.9 Feb 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants