-
-
Notifications
You must be signed in to change notification settings - Fork 93
Conversation
The old PR had some sync problems #499 (comment) |
@@ -16,15 +16,15 @@ | |||
} | |||
], | |||
"require": { | |||
"php": "^5.6 || ^7.0", | |||
"php": "^7.1", |
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.
Not sure if we can drop that on next minor release.
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.
We can, and did on all other packages.
"doctrine/phpcr-bundle": "^1.1", | ||
"doctrine/phpcr-odm": "^1.3 || ^2.0", | ||
"sonata-project/admin-bundle": "^3.21.0", | ||
"sonata-project/block-bundle": "^3.0", | ||
"symfony-cmf/resource-rest-bundle": "^1.0", | ||
"symfony-cmf/tree-browser-bundle": "^2.0", | ||
"symfony/framework-bundle": "^2.8 || ^3.0", | ||
"symfony/property-access": "^2.8 || ^3.0" | ||
"symfony/framework-bundle": "^2.8 || ^3.2", |
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 keeping 2.8 then, LTS?
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.
Yes, LTS
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.
looks good to me, thanks.
looks like the .travis.yml has already been updated to drop old versions, so we don't test them anymore. we should not claim to support them anymore.
@ElectricMaxxx please review again |
Thanks @core23 ! |
I am targeting this branch, because this is BC.
Changelog