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

Upgrade frontend to Ember 3.0.0 #1277

Closed
wants to merge 1 commit into from
Closed

Upgrade frontend to Ember 3.0.0 #1277

wants to merge 1 commit into from

Conversation

CvX
Copy link
Contributor

@CvX CvX commented Mar 24, 2018

Upgrades Ember to version 3.0.0.

@Turbo87
Copy link
Member

Turbo87 commented Mar 25, 2018

As mentioned on the Ember.js Slack, I would prefer it if the PR could be broken apart into multiple PRs to make review a little easier. My suggestion would be to start with the new testing API changes, then the native-dom-helpers and finally the upgrade itself.

package.json Outdated
@@ -46,6 +46,7 @@
"ember-cli-shims": "^1.2.0",
"ember-cli-sri": "2.1.1",
"ember-cli-uglify": "^2.0.0",
"ember-cli-update": "^0.15.0",
Copy link
Member

Choose a reason for hiding this comment

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

I'd prefer to use the global installation of this, since we don't need to install it for CI systems etc.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Alright, commit removed.

@CvX CvX force-pushed the ember-3 branch 5 times, most recently from 22db619 to 2f6d387 Compare April 1, 2018 15:10
Done using ember-cli-update.
Doesn’t enable new ember eslint rules for now.
@Turbo87
Copy link
Member

Turbo87 commented Apr 2, 2018

@CvX could you extract the ESLint changes to a separate PR too? Ideally all that remains in this PR will be the ember-source version bump :)

@CvX CvX closed this Apr 13, 2018
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.

None yet

2 participants