Skip to content

Fix allow_failures on Travis CI#2106

Merged
dunglas merged 1 commit into
api-platform:2.3from
teohhanhui:fix/travis-allow-failures
Jul 16, 2018
Merged

Fix allow_failures on Travis CI#2106
dunglas merged 1 commit into
api-platform:2.3from
teohhanhui:fix/travis-allow-failures

Conversation

@teohhanhui

Copy link
Copy Markdown
Contributor
Q A
Bug fix? yes (CI)
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets N/A
License MIT
Doc PR N/A

Comment thread .travis.yml
- APP_ENV=test

matrix:
jobs:

@teohhanhui teohhanhui Jul 16, 2018

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Minor future-proofing:

matrix is an alias of jobs

travis-ci/travis-ci#7754 (comment)

Travis CI docs are outdated: travis-ci/docs-travis-ci-com#1500 (comment)

@teohhanhui
teohhanhui force-pushed the fix/travis-allow-failures branch from 48a07a1 to aad9c6b Compare July 16, 2018 14:36
Comment thread .travis.yml
allow_failures:
env: SYMFONY_DEPRECATIONS_HELPER=0
- env: SYMFONY_DEPRECATIONS_HELPER=0
fast_finish: true

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Doesn't seem to have any effect at the moment... lol

@dunglas

dunglas commented Jul 16, 2018

Copy link
Copy Markdown
Member

Thanks @teohhanhui

@teohhanhui
teohhanhui deleted the fix/travis-allow-failures branch July 16, 2018 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants