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

Remove "<2.7" constraint on symfony/console #8151

Merged
merged 1 commit into from
Feb 8, 2017
Merged

Remove "<2.7" constraint on symfony/console #8151

merged 1 commit into from
Feb 8, 2017

Conversation

nicolas-grekas
Copy link
Contributor

@nicolas-grekas nicolas-grekas commented Jan 16, 2017

Sibling to magento/composer#3:

I guess this constraints was added before Symfony silenced its deprecation notices.
But now that it does, there should be no reason to exclude 2.7 & 2.8,
unless you've found a BC break blocker. In which case we'd be happy to know which one!

@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Jan 16, 2017

CLA assistant check
All committers have signed the CLA.

@ciaranmcnulty
Copy link

This has been preventing PhpSpec 3 being used with Magento 2, and means Magento 2 cannot be used with any currently supported Symfony components

@@ -43,7 +43,7 @@
"tubalmartin/cssmin": "2.4.8-p4",
"magento/magento-composer-installer": ">=0.1.11",
"braintree/braintree_php": "3.7.0",
"symfony/console": "~2.3 <2.7",
Copy link
Contributor

@okorshenko okorshenko Feb 3, 2017

Choose a reason for hiding this comment

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

In order to avoid issue with depreciation exceptions from Symfony code I'd recommend change this dependency in the following way:

"symfony/console": "~2.3, !=2.7.0",

This issue was introduced in 2.7.0 and was fixed in 2.7.1

bug #14900 Silence deprecation warnings by default (reecefowell)

So we would like to avoid using of problematic version
Thank you for your efforts!

okorshenko pushed a commit to magento-engcom/composer that referenced this pull request Feb 7, 2017
…: Remove "<2.7" constraint on symfony/console
okorshenko pushed a commit to magento-engcom/composer that referenced this pull request Feb 7, 2017
…: Remove "<2.7" constraint on symfony/console

 - updated minor version of magento/composer due to new dependency on symfony/console
@mmansoor-magento mmansoor-magento merged commit c6ef98f into magento:develop Feb 8, 2017
@okorshenko
Copy link
Contributor

@nicolas-grekas your pull request successfully merged to develop branch. Thank you for your contribution!

@okorshenko okorshenko added this to the February 2017 milestone Feb 11, 2017
magento-devops-reposync-svc pushed a commit that referenced this pull request Mar 14, 2023
Arrows Team - Bugfix delivery
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.

6 participants