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

Fix ember-canary related issue in tests #1080

Merged
merged 3 commits into from
Aug 4, 2021
Merged

Conversation

stefanpenner
Copy link
Member

Ember.DefaultResolver is no more, so rather trying to use it, we will simply rely on the basic resolver protocol. That way, not concrete object is needed, and we are future proof until such time as that changes.

stefanpenner and others added 2 commits August 3, 2021 16:44
Ember.DefaultResolver is no more, so rather trying to use it, we will simply rely on the basic resolver protocol. That way, not concrete object is needed, and we are future proof until such time as that changes.
@stefanpenner stefanpenner force-pushed the fix-resolver-extend branch 2 times, most recently from f4c7998 to 638b376 Compare August 4, 2021 00:00
Ember.DefaultResolver is no more, so rather trying to use it, we will
simply rely on the basic resolver protocol. That way, not concrete
object is needed, and we are future proof until such time as that
changes.
@rwjblue rwjblue merged commit 1a0b246 into master Aug 4, 2021
@rwjblue rwjblue deleted the fix-resolver-extend branch August 4, 2021 00:13
@rwjblue rwjblue changed the title Fix ember-canary related issue Fix ember-canary related issue in tests Aug 6, 2021
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