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

Remove ember-data from test harness #1069

Merged
merged 1 commit into from
Jul 20, 2021
Merged

Remove ember-data from test harness #1069

merged 1 commit into from
Jul 20, 2021

Conversation

rwjblue
Copy link
Member

@rwjblue rwjblue commented Jul 20, 2021

We no longer have moduleForModel tests (:tada:), so we don't need ember-data as a dependency any longer. This helps fix a number of issues with our test suite running against canary builds, where the older version of ember-data we were using was missing some required properties of the deprecate invocation (but upgrading to newer ember-data wasn't possible because of node engine support).

We no longer have `moduleForModel` tests (:tada:), so we don't need
ember-data as a dependency any longer. This helps fix a number of issues
with our test suite running against canary builds, where the older
version of ember-data we were using was missing some required properties
of the `deprecate` invocation (but upgrading to newer ember-data wasn't
possible because of node engine support).
@rwjblue rwjblue merged commit ee88968 into master Jul 20, 2021
@rwjblue rwjblue deleted the remove-ember-data branch July 20, 2021 21:56
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.

None yet

1 participant