[PR #15] Fix examples #52

New Issue

admin · 2026-01-23T11:35:02+01:00

admin commented

2026-01-23 11:35:02 +01:00

Original Pull Request: https://github.com/doctrine/KeyValueStore/pull/15

State: closed
Merged: Yes

The current example requires two additional use statements to be functional with a copy-paste. Additionally the composite key does not work as provided, it requires the key field names to be specified to function.

Not sure what @beberlei thinks about including in the example loading of the AnnotationRegistry? If this is done the example is a straight drop in and will work out of the box. If that's something you want added let me know.

**Original Pull Request:** https://github.com/doctrine/KeyValueStore/pull/15 **State:** closed **Merged:** Yes --- The current example requires two additional use statements to be functional with a copy-paste. Additionally the composite key does not work as provided, it requires the key field names to be specified to function. Not sure what @beberlei thinks about including in the example loading of the AnnotationRegistry? If this is done the example is a straight drop in and will work out of the box. If that's something you want added let me know.

admin added the pull-request label 2026-01-23 11:35:02 +01:00

admin closed this issue

2026-01-23 11:35:02 +01:00

admin referenced this issue

2026-01-23 11:35:07 +01:00

[PR #52] [MERGED] composer: use PSR-4 autoload #77

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/KeyValueStore#52