Open
Description
"WikibaseEntityStore" is already used by WMDE. Having the same name, esp for a project that does something different, is not ideal.
Perhaps more problematic is the clash of namespaces. The WMDE component uses Wikibase\EntityStore
, and so does this one. Generally you should not use a vendor namespace (ie Wikibase) if you are not that vendor. This was done right by using ppp/wikibase-entity-store
as package name. You can do something similar for the namespace, and go with PPP/WikibaseEntityStore
.
Not sure "EntityStore" is a good name to begin with here. To me this implies storage and retrieval of entities, while this component is about queries.
Metadata
Metadata
Assignees
Labels
No labels