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.merge deprecations #84

Merged
merged 1 commit into from
Apr 16, 2016
Merged

Conversation

trentmwillis
Copy link
Member

Addressing #79.

@dgeb dgeb merged commit 44d74a4 into ember-engines:master Apr 16, 2016
@dgeb
Copy link
Member

dgeb commented Apr 16, 2016

Thanks again @trentmwillis!

@rwjblue
Copy link
Member

rwjblue commented Apr 16, 2016

FYI - This deprecation has been removed from canary/beta/release and will be released as 2.5.1 sometime Monday.

@trentmwillis
Copy link
Member Author

@rwjblue interesting, looking at the PR to turn it off, sounds like it will still be turned back on in a future release though, correct? The ecosystem just isn't ready for it to be deprecated right now.

@rwjblue
Copy link
Member

rwjblue commented Apr 16, 2016

@trentmwillis - Yep, I was definitely not suggesting that this PR was invalid or anything. I'm mostly just pointing out, that many many things are spewing this deprecation at the moment so we decided to disable.

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.

None yet

3 participants