-
-
Notifications
You must be signed in to change notification settings - Fork 42
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
Use fixed php version #231
Conversation
project/.travis/install_test.sh
Outdated
@@ -5,9 +5,9 @@ mkdir --parents "${HOME}/bin" | |||
|
|||
# PHPUnit install | |||
if [ ${TRAVIS_PHP_VERSION} '<' '5.6' ]; then | |||
PHPUNIT_PHAR=phpunit-old.phar | |||
PHPUNIT_PHAR=phpuniti-4.8.9.phar |
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.
Do we need the patch level? phpunit-4.8.phar
should work too
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.
It's not listed, but indeed, there's a nice 302.
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.
fixed
project/.travis/install_test.sh
Outdated
else | ||
PHPUNIT_PHAR=phpunit.phar | ||
PHPUNIT_PHAR=phpunit-5.7.9.phar |
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.
Same here
What about adding phpunit as a dependency #197 ...? |
0f97e8d
to
ac55376
Compare
You mean using simple-phpunit? I thought about it, but right now I can't (too long), so here is a quick fix before I look into it. |
project/.travis/install_test.sh
Outdated
@@ -5,9 +5,9 @@ mkdir --parents "${HOME}/bin" | |||
|
|||
# PHPUnit install | |||
if [ ${TRAVIS_PHP_VERSION} '<' '5.6' ]; then | |||
PHPUNIT_PHAR=phpunit-old.phar | |||
PHPUNIT_PHAR=phpuniti-4.8.phar |
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.
Typo here: phpuniti
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.
Good catch!
We don't want to use phpunit 6 yet.
ac55376
to
5bd119f
Compare
The commit message was wrong. It's not about |
I'm very tired, sorry… |
I hope there are no other fuck ups… |
Didn't see that either... |
I'm trying out simple-phpunit now if anyone's interested sonata-project/SonataAdminBundle#4307 |
We don't want to use phpunit 6 yet.