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

Use installed PHPUnit binary #456

Closed
wants to merge 1 commit into from

Conversation

theofidry
Copy link
Contributor

Using Travis one is useful when supporting different versions of PHP, as for example PHPUnit 5.2 is not supported on PHP 5.3 or 5.4. But in our case we don't really care and we are installing PHPUnit anyway, so we might as well use it directly instead of having to update Travis one.

Using Travis one is usefull when supporting different versions of PHP, as for example PHPUnit 5.2 is not supported on PHP 5.3 or 5.4. But in our case we don't really care and we are installing PHPUnit anyway, so we might as well use it directly instead of having to update Travis one.
@theofidry
Copy link
Contributor Author

My bad we're not requiring phpunit in our package.json...

@theofidry theofidry closed this Mar 12, 2016
@teohhanhui
Copy link
Contributor

This is not currently possible due to conflicts. This is also the reason why some unit tests are marked as skipped.

@theofidry
Copy link
Contributor Author

Which conflicts are you referring too?

@teohhanhui
Copy link
Contributor

See #433

We need phpspec/prophecy to support phpdocumentor/reflection-docblock v3, and of course phpunit/phpunit updated to that version of Prophecy.

@theofidry
Copy link
Contributor Author

We should have an issue for it in the backlog to keep track of it then :/

@theofidry theofidry deleted the travis-phpunit branch March 13, 2016 12:48
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.

None yet

2 participants