django-rest-marshmallow
django-rest-marshmallow copied to clipboard
PIP dependencies don't seem to list marshmallow.
Installing this module via poetry/pip didn't pull down marshmallow, so I think you're missing a dependency there.
Looks like rest_framework is also missing there.
Indeed, this package declares no explicit dependencies in install_requires, and that seems to have been an explicit decision from the beginning (correct me if I'm wrong @tomchristie ). My guess is that django and marshmallow are considered "peer dependencies" that the consumer is responsible for installing themselves.
In Python packaging, though, everything is a peer dependency, so I think it would be fine to add both django and marshmallow as explicit dependencies in setup.py .
I guess I'm of the opinion that once I install a package in Python, I should at least be able to import it without fatal errors. Optional dependencies are one thing, but if a package won't even import at all, that feels like the case for a explicit dependency to me.
(Especially as the docs for this package do mention specific versions you require.)
Ok, let’s go ahead and add them. Would you like to send a PR?