-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Updated Travis CI configuration #171
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
Conversation
Seems |
It makes sense |
Closing it as "won't fix". |
@javiereguiluz my point in the comment you linked is that it does not really make sense to test against different deps versions as we are locking deps |
@stof you are right. Sorry for not understanding you earlier. I've just reopened and made a lot of simplifications to the Travis config. Thanks! |
- php: hhvm | ||
- php: nightly | ||
allow_failures: | ||
- php: hhvm | ||
- php: nightly | ||
- env: SYMFONY_VERSION="2.7.x-dev" | ||
- env: SYMFONY_VERSION="2.8.x-dev" | ||
|
||
before_install: | ||
- composer self-update |
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.
you can even get rid of this (it was necessary to have a Composer version supporting --prefer-lowest
because Travis has not updated its environment since February)
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.
Done. Thanks!
Cool! It has become much easier now 👍 |
@bocharsky-bw I agree! @stof's comments are always spot on. I love the new Travis config. |
@javiereguiluz I also suggest enabling the Travis caching for the composer cache, so that it does not to re-download the same archives for each build (project with locked dependencies benefit from it even more than others because the deps change only when you commit a change, not when a new release of a deps is done: cache:
directories:
- $HOME/.composer/cache/files |
Nice! Enabled the cache too. Thanks. |
👍 |
Changes made as recommended by @stof in #154 (comment)