-
-
Notifications
You must be signed in to change notification settings - Fork 69
Minor composer improvements #200
Minor composer improvements #200
Conversation
@@ -36,7 +36,9 @@ | |||
"jackalope/jackalope-doctrine-dbal": "^1.0", | |||
"php-mock/php-mock": "^1.0", | |||
"predis/predis": "^0.8 || ^1.0", | |||
"symfony/phpunit-bridge": "^3.3.12 || ^4.0" | |||
"symfony/console": "^2.8 || ^3.2 || ^4.0", | |||
"symfony/framework-bundle": "^2.8 || ^3.2 || ^4.0", |
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.
Why don't we need this on the require section?
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.
On master it is on require-dev section already. Added here: #161. This is only used for the commands. Not sure if the intention is to have commands only optionals. With this we do not really change what gets installed, because other dependencies on require-dev already use framework-bundle.
We can maybe think if this should be required o require-dev on the pull request where we add commands as a services.
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.
Okay for now
Thanks @jordisala1991 ! |
I am targeting this branch, because this is BC.
Subject
This bundle also needs: