-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Support for Doctrine ORM >=2.5 #3726
Conversation
Could you update other composer.jsons? https://github.com/Sylius/Sylius/search?l=json&q=doctrine%5C%2Form&type=Code&utf8=%E2%9C%93 |
@pamil does the fix work on |
@@ -22,7 +22,7 @@ | |||
"require": { | |||
"php": ">=5.5.9", | |||
|
|||
"doctrine/orm": "2.4.*", | |||
"doctrine/orm": "^2.4.8", |
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.
Channel really requires Doctrine ORM? Should not be the case ...
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 like ORM's exceptions are used in PhpDocs in ChannelRepositoryInterface
.
doctrine/common
is used too in ChannelsAwareInterface
only.
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.
ORM's exceptions should be removed from interface docs. The interface doesn't know anything about the implementation especially about the thrown exceptions.
Let's require doctrine/common
then.
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.
I opened a separate issue for that #3729.
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.
👍
@michalmarcinkowski it should be fine on all |
Support for Doctrine ORM >=2.5
Thanks Kamil! |
Support for Doctrine ORM >=2.5
No description provided.