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 deprecation Ember.merge for 2.5.0 #148

Merged
merged 1 commit into from
Mar 30, 2016

Conversation

gertjanwytynck
Copy link
Contributor

DEPRECATION: Usage of Ember.merge is deprecated, use Ember.assign instead. [deprecation id: ember-metal.merge]

More info: emberjs/ember.js#12303

Setup:

Ember             : 2.5.0-beta.3
Ember Data        : 2.5.0-beta.3
jQuery            : 2.2.1
Ember Simple Auth : 1.1.0-beta.3

@jpadilla
Copy link
Contributor

I don't think our tests cover 2.5.0 beta or at least I can't see the deprecation anywhere on Travis, but this seems like a small enough change to accept. @eluciano11 thoughts?

@eluciano11
Copy link
Contributor

No, currently we don't cover the 2.5.0 scenario in our ember try config. We currently test against every version from 1.11 through 2.0.0. We can change that to make tests against ember 1.13 through 2.5. I can make those changes and add them to this PR.

@eluciano11
Copy link
Contributor

@homu r+

@homu
Copy link
Contributor

homu commented Mar 30, 2016

📌 Commit 4b0a00b has been approved by eluciano11

homu added a commit that referenced this pull request Mar 30, 2016
fix deprecation Ember.merge for 2.5.0

DEPRECATION: Usage of `Ember.merge` is deprecated, use `Ember.assign` instead. [deprecation id: ember-metal.merge]

More info: emberjs/ember.js#12303

Setup:

```
Ember             : 2.5.0-beta.3
Ember Data        : 2.5.0-beta.3
jQuery            : 2.2.1
Ember Simple Auth : 1.1.0-beta.3
```
@homu
Copy link
Contributor

homu commented Mar 30, 2016

⌛ Testing commit 4b0a00b with merge 146b4bc...

@homu
Copy link
Contributor

homu commented Mar 30, 2016

☀️ Test successful - status

@homu homu merged commit 4b0a00b into fenichelar:master Mar 30, 2016
@eluciano11
Copy link
Contributor

@gertjanwytynck Thank you for your contribution!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants