-
-
Notifications
You must be signed in to change notification settings - Fork 1
Updated PHPCR console class names #5
base: master
Are you sure you want to change the base?
Conversation
new \Doctrine\ODM\PHPCR\Tools\Console\Command\RegisterSystemNodeTypesCommand(), | ||
new \Jackalope\Tools\Console\Command\JackrabbitCommand(), | ||
new \Jackalope\Tools\Console\Command\InitDoctrineDbalCommand(), | ||
new \Jackalope\Tools\Console\Command\InitDoctrineDbalCommand() |
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.
please keep the trailing comma
can you also fix |
"doctrine/phpcr-odm": "dev-master", | ||
"jackalope/jackalope-jackrabbit": "dev-master", | ||
"jackalope/jackalope-doctrine-dbal": "dev-master", | ||
"doctrine/phpcr-odm": "1.0.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.
no need to be so strict on the version numbers
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.
@lsmith77 we need to rely on some internal factories APIs because of how Zend\Di
handles introspection on them - do you think it is safe to assume 1.0.*
on each of those?
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.
very much so ..
btw we will soon wrap up 1.1 releases of both .. might make sense to test current master versions too and if things work also allow 1.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.
I don't mind changing it, but are you thinking 1.0.* or 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.
@milan 1.0.*
- we can bump later on.
Noticed the class names had changed in PHPCR which was erroring for me