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

Update ember and ember-data to 2.17. Fix deprecations. #207

Merged
merged 8 commits into from
Dec 4, 2017

Conversation

m-basov
Copy link
Contributor

@m-basov m-basov commented Dec 3, 2017

Hi @dustinfarris! I did upgrade to addon with ember-cli-update. Fixed deprecations. Could you test and merge it if all works well?

Closes #205 and #206

UPD. Tests are passing locally but I am trying to fix them at CI.

UPD. @dustinfarris could you check CI settings please? Seems some very old image is used. Ping me so we can resolve issue together.

@dustinfarris
Copy link
Owner

@kolybasov thank you so much for this!

You're right, CircleCI was running on a Ubuntu 12.04 image that doesn't come with yarn. I updated the setting to 14.04 (with yarn) and made a slight modification to circle.yml based on their docs.

@m-basov
Copy link
Contributor Author

m-basov commented Dec 4, 2017

@dustinfarris thanks for the fix! Let me know if I need to made some changes to merge this.

@dustinfarris dustinfarris merged commit 7231937 into dustinfarris:master Dec 4, 2017
@dustinfarris
Copy link
Owner

@kolybasov all good! really can't thank you enough for putting time into getting this done. 💙💙💙

will do a major version release since the supported versions of Ember have changed

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.

Upgrade ember-cli
2 participants