Skip to content

fix: updated the ember-cli version to the latest release #80

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

Merged
merged 1 commit into from
Dec 10, 2015

Conversation

Brocco
Copy link
Contributor

@Brocco Brocco commented Nov 27, 2015

Fixes #73 by updating the ember-cli version to the latest release

https://github.com/ember-cli/ember-cli/releases/tag/v1.13.13

Minor build issue fixed via helper update to mock-ui.js (pulled directly from ember's helper class)

@@ -35,7 +35,7 @@
"broccoli-merge-trees": "^0.2.1",
"broccoli-writer": "^0.1.1",
"chalk": "^1.1.0",
"ember-cli": "ember-cli/ember-cli#ae5e72614e40f5e6dd57463e526c65da899fde07",
"ember-cli": "ember-cli/ember-cli#ef6b848b38d85037f917dcc14a6735df59a94ea7",
Copy link
Member

Choose a reason for hiding this comment

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

can we use the version number instead of the commit hash?

@Brocco
Copy link
Contributor Author

Brocco commented Nov 28, 2015

I'm fine with doing that, we can even pull in the latest version via npm directly instead of via github?

  • github via version/tag
  • npm w/ exact version

Either way will work just fine, and the necessary coding changes in the repo are already handled.

@cironunes
Copy link
Member

Awesome! Please amend the commit changing to the exact version and it will be good to merge.

@Brocco Brocco force-pushed the issue-73-ember-cli-version branch from dabb431 to 447a63a Compare November 28, 2015 17:31
@Brocco
Copy link
Contributor Author

Brocco commented Nov 28, 2015

This also now includes the change that fixes a unit test failure issue

@cironunes
Copy link
Member

@Brocco the best would be having them in two different commits. Can you just remove the change and fix the commit message?

@Brocco Brocco force-pushed the issue-73-ember-cli-version branch from 34dca41 to 09543a7 Compare December 3, 2015 02:59
@IgorMinar
Copy link
Contributor

lots of test failures. @Brocco can you please investigate?

@IgorMinar
Copy link
Contributor

ping

@Brocco Brocco force-pushed the issue-73-ember-cli-version branch from 09543a7 to 9c66c5f Compare December 10, 2015 14:36
Change the reference to ember-cli from a github reference to an npm ref via fixed version
Add errorStream configuration in mock-ui test helper

fixes angular#73
@Brocco Brocco force-pushed the issue-73-ember-cli-version branch from 9c66c5f to 10eae9e Compare December 10, 2015 15:09
@Brocco
Copy link
Contributor Author

Brocco commented Dec 10, 2015

Node v5 checks were failing had to update some broccoli package references, all is good now

@cironunes cironunes merged commit 10eae9e into angular:master Dec 10, 2015
@cironunes
Copy link
Member

ci green, merged. Thanks @Brocco

@Brocco Brocco deleted the issue-73-ember-cli-version branch December 10, 2015 15:37
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update ember-cli version
3 participants