Skip to content

Commit 8666b66

Browse files
minor #78 Fix Travis SYMFONY_REQUIRE config (wouterj)
This PR was merged into the master branch. Discussion ---------- Fix Travis SYMFONY_REQUIRE config The `composer install` script was running in `before_install`, while the `SYMFONY_REQUIRE` + symfony/flex was configured in `install`. Swapping these two from position fixes the Travis setup. (flex has to be configured before `composer install`). I also removed `if ! [ -v "$DEPENDENCIES" ]; then composer require --no-update ${DEPENDENCIES}; fi;`, it seems to be a left over from previously used configs as the `DEPENDENCIES` is not defined anywhere. Commits ------- 8143c62 Swap install scripts from before_install & install
2 parents 5e1cab3 + 8143c62 commit 8666b66

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

.travis.yml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,16 +28,15 @@ matrix:
2828
before_install:
2929
- if [[ $COVERAGE != true ]]; then phpenv config-rm xdebug.ini || true; fi
3030
- if ! [ -z "$STABILITY" ]; then composer config minimum-stability ${STABILITY}; fi;
31-
- if ! [ -v "$DEPENDENCIES" ]; then composer require --no-update ${DEPENDENCIES}; fi;
32-
- composer install
33-
34-
install:
3531
- |
3632
# Install symfony/flex
3733
if [[ $deps = low ]]; then
3834
export SYMFONY_REQUIRE='>=2.3'
3935
fi
4036
composer global require --no-progress --no-scripts --no-plugins symfony/flex dev-master
37+
38+
install:
39+
- composer install
4140
- ./vendor/bin/simple-phpunit install
4241

4342
script:

0 commit comments

Comments
 (0)