-
Notifications
You must be signed in to change notification settings - Fork 51
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
WIP: upgrade dependencies #120
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ember-cli@3.15.1 is out, mind updating for that blueprint?
3960d8e
to
261012a
Compare
Do not merge. CI is green even so one scenario has failed:
Seems like the hacky work-a-round used by |
@@ -19,50 +19,55 @@ | |||
"start": "ember serve", | |||
"test": "ember test", | |||
"test:all": "ember try:each", | |||
"test:node": "for i in node-tests/*/*; do mocha $i; done" | |||
"test:node": "( for i in node-tests/*/*; do mocha $i || exit $?; done )" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exit code of a for loop is the exit code of last iteration. So that's not very helpful here. To avoid wrong test result we use a subshell that is exit as soon as one test fails. Have found that trick here: https://stackoverflow.com/questions/14059342/how-to-get-the-exit-status-a-loop-in-bash
This will stop the tests as soon as the first file fails. It will not process later tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch!
3725f4e
to
8e1f145
Compare
601180f
to
6a19a8d
Compare
b4a8200
to
ad314c7
Compare
I was able to trace it down to the upgrade of
This error is not caused by this addon but by the browser. It's somehow forwarded to Ember CLI and causes |
Fear I have broken GitHub / Travis integration by rewriting the history too much. The commits are not linked to correct CI runs. I'm also still struggling getting a clear picture about which upgrade has broken the pipeline... Please just ignore the PR for now until I have more details to share. |
Superseded by #126 |
Upgrades all dependencies to their latest versions.