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

Updating symfony/phpunit-bridge version constraint #947

Closed
wants to merge 2 commits into from

Conversation

Jean85
Copy link
Contributor

@Jean85 Jean85 commented Feb 24, 2016

I recently installed Symfony 3.0.2, done a composer update and discovered (using vinkla/climb) that symfony/phpunit-bridge couldn't be updated to the latest major version. This should make the upgrade immediately executable on a fresh install.

I didn't update the lock because v3 increases the minimum PHP version to 5.5.9, so this remains backward compatible with older PHP versions.

@@ -24,7 +24,7 @@
},
"require-dev": {
"sensio/generator-bundle": "^3.0",
"symfony/phpunit-bridge": "^2.7"
"symfony/phpunit-bridge": "^2.7|^3.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

What about to use "symfony/phpunit-bridge": "^3.0" for Symfony 3.0?

Copy link
Contributor

Choose a reason for hiding this comment

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

+1 here we are on the master branch so no need to comply with 2.x branch.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right! Revising PR now, ^2.7 is needed when backporting..

@Jean85
Copy link
Contributor Author

Jean85 commented Feb 24, 2016

Since we are on master and with SF3.x, I updated this PR to request phpunit-bridge ^3.0. ^2.7 is needed when backporting to older branches

@Tobion
Copy link
Contributor

Tobion commented Feb 25, 2016

👍 Could be merged in 3.0

@xabbuh
Copy link
Member

xabbuh commented Feb 25, 2016

👍

@fabpot
Copy link
Member

fabpot commented Feb 26, 2016

Thank you @Jean85.

@fabpot fabpot closed this in c2c19a2 Feb 26, 2016
@Jean85
Copy link
Contributor Author

Jean85 commented Feb 26, 2016

You're welcome @fabpot !
But... Shouldn't the composer.lock be updated now? And with which PHP version? The lowest? I will happily PR that.

@stof
Copy link
Member

stof commented Feb 26, 2016

@Jean85 we generally update run composer update before each release (after the symfony/symfony release)

@Jean85
Copy link
Contributor Author

Jean85 commented Feb 26, 2016

Ok great!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants