Skip to content
This repository has been archived by the owner on Mar 30, 2018. It is now read-only.

Improve Travis matrix #207

Merged
merged 14 commits into from
Apr 21, 2017
Merged

Improve Travis matrix #207

Changes from 4 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
adcd387
Improve deps testing, add php 7.1, php7 must pass tests
snapshotpl Mar 4, 2017
fbc92b7
Disable xdebug and check coverage once to improve test time
snapshotpl Mar 4, 2017
3d3e76a
xdebug must be enabled
snapshotpl Mar 4, 2017
9d914a9
Simplify travis
snapshotpl Apr 13, 2017
7377007
Improve composer
snapshotpl Apr 14, 2017
5202f15
Merge branch 'master' into improve-testing
snapshotpl Apr 18, 2017
e755a06
Merge branch 'master' into improve-testing
snapshotpl Apr 20, 2017
e1e9a85
Use matrix for env
snapshotpl Apr 20, 2017
b64f9bd
Fix travis
snapshotpl Apr 20, 2017
ae2bbe8
Fix travis
snapshotpl Apr 20, 2017
c18a8a9
Fix composer
snapshotpl Apr 20, 2017
495f5b7
Exclude php 7.1 without coverage
snapshotpl Apr 20, 2017
8b605cd
Small improvements
snapshotpl Apr 20, 2017
d332be9
Add phpunit 5.0 compatibility
snapshotpl Apr 20, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
40 changes: 24 additions & 16 deletions .travis.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,21 +1,32 @@
language: php
sudo: false

php:
- 7.0
- 5.6
- 5.5
- 5.4
- hhvm
- hhvm-nightly

matrix:
include:
- php: 5.4
Copy link
Collaborator

Choose a reason for hiding this comment

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

Per #226, we can remove the EOL php versions 5.3 to 5.5.

env: COMPOSER_FLAGS='--prefer-lowest'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would it be simpler to just add an env block later? e.g.

env:
  matrix:
    - COMPOSER_FLAGS="--prefer-lowest"
    - COMPOSER_FLAGS=""

Keeps the list of flags and versions distinct?

- php: 5.4
Copy link
Member

Choose a reason for hiding this comment

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

I think you can skip the flags here as it's the default behaviour

- php: 5.5
env: COMPOSER_FLAGS='--prefer-lowest'
- php: 5.5
- php: 5.6
env: COMPOSER_FLAGS='--prefer-lowest'
- php: 5.6
Copy link
Member

Choose a reason for hiding this comment

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

versions should be strings

- php: 7.0
env: COMPOSER_FLAGS='--prefer-lowest'
- php: 7.0
- php: 7.1
env: COMPOSER_FLAGS='--prefer-lowest'
- php: 7.1
Copy link
Member

Choose a reason for hiding this comment

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

we're missing a nightly build that should be allowed to fail

env:
- TEST_COVERAGE=true
- php: hhvm
env: COMPOSER_FLAGS='--prefer-lowest'
- php: hhvm
fast_finish: true
allow_failures:
- php: 7.0
- php: hhvm
- php: hhvm-nightly
Copy link
Collaborator

Choose a reason for hiding this comment

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

There's no hhvm-nightly build now :)


branches:
only:
- master
Expand All @@ -24,12 +35,9 @@ cache:
directories:
- $HOME/.composer/cache

before_install:
- travis_retry composer self-update

before_script:
- travis_retry composer install --no-interaction
- composer update --no-interaction --prefer-source $COMPOSER_FLAGS
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there some reason to put --prefer-source here? Installing source packages takes more time than dist ones.


script:
- vendor/bin/phpunit --coverage-text
- if [[ $TEST_COVERAGE == 'true' ]]; then vendor/bin/phpunit --coverage-text ; else vendor/bin/phpunit ; fi
Copy link
Member

Choose a reason for hiding this comment

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

if [ -n "$TEST_COVERAGE" ]; then?

Copy link
Member

Choose a reason for hiding this comment

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

Single brackets are also enough AFAIK

Copy link
Collaborator

Choose a reason for hiding this comment

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

What do you think about introducing $PHPUNIT_FLAGS? This way we can remove if from this statement.

- vendor/bin/behat