Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Declare sphinx-me as an extra dependency #29

Open
aaugustin opened this issue Jul 18, 2016 · 2 comments
Open

Declare sphinx-me as an extra dependency #29

aaugustin opened this issue Jul 18, 2016 · 2 comments

Comments

@aaugustin
Copy link
Contributor

Hello,

Most users of django-overextends don't need sphinx-me because they won't build the documentation.

https://packaging.python.org/requirements/#install-requires states that install_requires "should be used to specify what a project minimally needs to run correctly".

https://github.com/pypa/sampleproject/blob/cec59671175bc2c86563ac80c90ba28b068a1335/setup.py#L78-L91 suggests that you could put sphinx-me in extras_require instead of install_requires.

Thanks!

@aaugustin
Copy link
Contributor Author

I assume that sphinx-me is in setup.py to get an automagic setup. Its readme suggests that you could achieve the same effect with a requirements file.

@blag
Copy link

blag commented Mar 19, 2017

This is fixed in master but hasn't been released to PyPI yet. @stephenmcd If you have some time can you create a new PyPI release?

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

No branches or pull requests

2 participants