Skip to content
This repository was archived by the owner on Jan 29, 2020. It is now read-only.

Allow sebastian/version 2.0#21

Merged
weierophinney merged 1 commit intozendframework:masterfrom
remicollet:patch-2
Sep 6, 2016
Merged

Allow sebastian/version 2.0#21
weierophinney merged 1 commit intozendframework:masterfrom
remicollet:patch-2

Conversation

@remicollet
Copy link
Copy Markdown
Contributor

There is no API change in V2, only a raise of the PHP minimal version to 5.6

There is no API change in V2, only a raise of the PHP minimal version to 5.6
@weierophinney
Copy link
Copy Markdown
Member

The only reason for explicitly indicating the sebastian/version dependency is due to the way the test bootstrap works. We could likely solve this by requiring a minimum PHPUnit version of 4.8 instead. Can you give that a try, and run composer update --prefer-lowest and see if the tests run?

@remicollet
Copy link
Copy Markdown
Contributor Author

--prefer-lowest pull PHPUnit 4.8 and version 1.0.0

$ vendor/bin/phpunit 

This version of PHPUnit (release-2.6.1-7-g93608c3) is not supported in the zend-test unit tests. Supported is version 4.0.0 or higher. See also the CONTRIBUTING.md file in the component root.

So keeping min version for sebastian/version is still needed

Tests ok with phpunit 4.8.0 + version 1.0.4
Tests ok with phpunit 5.3.2 + version 2.0.0

@weierophinney weierophinney added this to the 3.0.2 milestone Sep 6, 2016
@weierophinney weierophinney self-assigned this Sep 6, 2016
@weierophinney weierophinney merged commit 93608c3 into zendframework:master Sep 6, 2016
weierophinney added a commit that referenced this pull request Sep 6, 2016
Allow sebastian/version 2.0
weierophinney added a commit that referenced this pull request Sep 6, 2016
weierophinney added a commit that referenced this pull request Sep 6, 2016
weierophinney added a commit that referenced this pull request Sep 6, 2016
@weierophinney
Copy link
Copy Markdown
Member

Thanks, @remicollet

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants