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

Updates for Ember 1.13. #14

Closed
wants to merge 2 commits into from

Conversation

rwjblue
Copy link
Contributor

@rwjblue rwjblue commented Jun 19, 2015

  • Only use Ember.View.views as a fallback if -view-registry:main is not available.
  • Remove assertion confirming that different elements are created upon .rerender() (a rerender in Ember 1.13 maintains the same element).
  • Update ember-try configuration for 1.13.
    • Change scenario names to match the default ones added by ember-cli
      (and used by ember-modal-dialog/ember-tether)
    • Make 1.13 the default version.

rwjblue added 2 commits June 18, 2015 21:59
* Change scenario names to match the default ones added by ember-cli
  (and used by ember-modal-dialog/ember-tether)
* Make 1.13 the default version.
* Only use `Ember.View.views` as a fallback if `-view-registry:main` is
  not available.
* Remove assertion confirming that different elements are created upon
  `.rerender()` (a rerender in Ember 1.13 maintains the same element).
@rwjblue rwjblue force-pushed the update-test-suite-for-1-13 branch from 465f0a6 to 319d844 Compare June 19, 2015 12:58
@rwjblue
Copy link
Contributor Author

rwjblue commented Jun 19, 2015

I'm not really sure why Ember 1.11.3 is tanking in Travis (seems like phantomjs just crashes). ember try:testall passes locally.

I'll try to play with it some more tonight/tomorrow.

@mike-north
Copy link

PhantomJS 1.9 could be the problem. Consider downloading 2.0

before_install:
  - mkdir travis-phantomjs
  - wget https://s3.amazonaws.com/travis-phantomjs/phantomjs-2.0.0-ubuntu-12.04.tar.bz2 -O $PWD/travis-phantomjs/phantomjs-2.0.0-ubuntu-12.04.tar.bz2
  - tar -xvf $PWD/travis-phantomjs/phantomjs-2.0.0-ubuntu-12.04.tar.bz2 -C $PWD/travis-phantomjs
  - export PATH=$PWD/travis-phantomjs:$PATH
  - "npm config set spin false"
  - "npm install -g npm@^2"

},
resolutions: {
"ember": "beta"
'ember': '1.11.3'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What are your thoughts on using tilde dependencies for testing previous release versions?

@krisselden
Copy link
Contributor

@rwjblue I totally didn't see this and redid it on master, I didn't however do the ember try stuff

@chrislopresto
Copy link
Contributor

#18 covered the ember-cli 1.13.1 upgrade and took care of the ember try stuff. I rebased and merged via #30. I stuck with @krisselden's approach to avoid use of the viewRegistry since it was already in master. I'll go ahead and close here.

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.

4 participants