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

Compatibility updates for Ember 2.12 #21

Merged
merged 18 commits into from May 3, 2017

Conversation

tzellman
Copy link
Collaborator

@tzellman tzellman commented May 3, 2017

This branch started as a base of your 2.9 compatibility branch.

The biggest item to note is that I had to skip 3 tests in r-get-test.js due to this issue: emberjs/ember.js#15013

Other than that, the updates are related to dependency bumps, as well as the change to support factoryFor.

taras and others added 14 commits September 18, 2016 15:58
Note that I cannot update to ember 2.12 just yet until there is a resolution for emberjs/ember.js#15013
For now, I am skipping the three tests that assert that errors are thrown, until I can find a resolution for emberjs/ember.js#15013
Additionally, I fixed the _lookupFactory deprecation in favor of factoryFor
Also, removed a few jshint pragmas
this requires adding ember-cli-string-helpers, since it was refactored out and tracked separately as of 2.x
@taras
Copy link
Member

taras commented May 3, 2017

This is awesome! Thank you so much!

I'm going to add you as a contributor to this repo so you can merge it when you think it's ready.

@tzellman
Copy link
Collaborator Author

tzellman commented May 3, 2017

Likewise! This addon is fantastic, and a staple for my apps.
I'll look into the CI failures first.

Since there were API changes (e.g. r/mut removal), I assume we should bump the version to 1.0.0 (or 0.5.0 if you would prefer).

@taras
Copy link
Member

taras commented May 3, 2017

This addon is fantastic, and a staple for my apps.

Very happy. Which helpers do you use?

Since there were API changes (e.g. r/mut removal), I assume we should bump the version to 1.0.0 (or 0.5.0 if you would prefer).

Yeah. good idea.

@tzellman
Copy link
Collaborator Author

tzellman commented May 3, 2017

Very happy. Which helpers do you use?

Primarily r in conjunction with ember-composable-helpers. r/log is also very useful.

Yeah. good idea.

1.0.0 or 0.5.0 ?

@taras
Copy link
Member

taras commented May 3, 2017

Let's go for 0.5.0. Can you please remove keep-latest helper because I don't think it's necessary with ember-concurrency in the picture.

@tzellman
Copy link
Collaborator Author

tzellman commented May 3, 2017

Sure thing.

@tzellman tzellman merged commit 0c0e880 into EmberSherpa:master May 3, 2017
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

2 participants