Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

PIP dependencies don't seem to list marshmallow. #138

Open
dragonpaw opened this issue Apr 6, 2020 · 4 comments
Open

PIP dependencies don't seem to list marshmallow. #138

dragonpaw opened this issue Apr 6, 2020 · 4 comments

Comments

@dragonpaw
Copy link

Installing this module via poetry/pip didn't pull down marshmallow, so I think you're missing a dependency there.

@dragonpaw
Copy link
Author

Looks like rest_framework is also missing there.

@sloria
Copy link
Member

sloria commented Apr 7, 2020

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 .

@dragonpaw
Copy link
Author

dragonpaw commented Apr 7, 2020

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.)

@sloria
Copy link
Member

sloria commented Apr 7, 2020

Ok, let’s go ahead and add them. Would you like to send a PR?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants