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

chore(ci): match Ember Data version in LTS tests #5

Merged
merged 6 commits into from
Mar 20, 2020

Conversation

alexlafroscia
Copy link
Contributor

@alexlafroscia alexlafroscia commented Mar 20, 2020

The 3.12 tests failed in #4 so I want to land this as a separate PR to establish a baseline that the current work on master actually does work in Ember 3.12.

It should, since the add-on seems to work in our own Ember app that uses Ember 3.12.

@alexlafroscia alexlafroscia force-pushed the use-lts-ember-data branch 4 times, most recently from 9a3eafb to 114fd56 Compare March 20, 2020 16:55
@alexlafroscia alexlafroscia force-pushed the use-lts-ember-data branch 2 times, most recently from 604bc78 to 5f4bd6f Compare March 20, 2020 19:15
@alexlafroscia alexlafroscia merged commit 83c0d77 into master Mar 20, 2020
@alexlafroscia alexlafroscia deleted the use-lts-ember-data branch March 20, 2020 20:11
@alexlafroscia
Copy link
Contributor Author

Huh, turns out that this was the underlying issue

emberjs/ember.js#18827

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.

2 participants