Skip to content
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

Update .travis.yml #347

Closed
wants to merge 3 commits into from
Closed

Update .travis.yml #347

wants to merge 3 commits into from

Conversation

andreybolonin
Copy link
Contributor

No description provided.

@bytehead
Copy link
Member

Seems not to work:

7.3.0RC1 is not pre-installed; installing
Downloading archive: https://s3.amazonaws.com/travis-php-archives/binaries/ubuntu/14.04/x86_64/php-7.3.0RC1.tar.bz2
$ curl -s -o archive.tar.bz2 $archive_url && tar xjf archive.tar.bz2 --directory /
bzip2: (stdin) is not a bzip2 file.
tar: Child returned status 2
tar: Error is not recoverable: exiting now

$ phpenv global 7.3.0RC1
rbenv: version `7.3.0RC1' not installed
The command "phpenv global 7.3.0RC1" failed and exited with 1 during .

@andreybolonin
Copy link
Contributor Author

andreybolonin commented Oct 13, 2018

@bytehead yep, you are right.

Waiting for php 7.3 release.

@daFish
Copy link

daFish commented Oct 14, 2018

The notation is just 7.3 - this works as of a couple of days.

@andreybolonin
Copy link
Contributor Author

Php 7.3 works in travis build matrix.
But waiting for merge KnpLabs/Gaufrette#587

@bytehead
Copy link
Member

bytehead commented Apr 3, 2019

PHP 7.3 is now available, but the tests still fail:

There was 1 error:
1) Oneup\UploaderBundle\Tests\Uploader\Chunk\Storage\GaufretteStorageTest::testExistanceOfTmpDir
"continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"?

Would you mind to fix this before merge? 🙂

@bytehead bytehead mentioned this pull request Jan 13, 2020
@bytehead
Copy link
Member

Closing in favor of #369

@bytehead bytehead closed this Jan 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants