-
Notifications
You must be signed in to change notification settings - Fork 59
Conversation
* or if all properties have to be set, even if only a subset of properties | ||
* changed. | ||
* | ||
* @return bool |
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.
you should use {@inheritDoc}
for all methods documented in the interface, to avoid duplicating all phpdoc. It makes it easier to maintain it (typos occurs only in 1 place for instance)
I've made some changes according @stof suggestions. |
@@ -3,6 +3,12 @@ | |||
"require": { | |||
"doctrine/common": "*" | |||
}, | |||
"suggest": { | |||
"riak/riak-client": "dev-master" |
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.
suggest does not expect a version constraint but a string displayed to the user. so you could make it better by using "riak/riak-client": "to use the Riak storage"
. This will output "doctrine/key-value-store" suggest installing "riak/riak-client" (to use the Riak storage)
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.
Ah awesome!
Thank you ;)
@beberlei It's mergeable? |
I added the riak storage.
It use the update from the offical php-driver.