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

Please allow using this project a library #294

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

oz123
Copy link

@oz123 oz123 commented Jan 13, 2022

This project is very useful also as a library. For example, pipenv uses it here:

https://github.com/pypa/pipenv/blob/4fa67914f7562db947e1b30875cf96a709e8f4f5/pipenv/core.py#L219

Because of the top level import of docopt, it must also pull docopt as a dependecy. It would be really nice if we
could remove this dependency.

oz123 added 2 commits January 13, 2022 11:16
This allows upstream projects (such as pipenv) to vendor
only this library, without the command line dependencies.
This is now documented in requirements-cli.txt,
and in setup.py in extras_require.
@oz123
Copy link
Author

oz123 commented Jan 13, 2022

Just noticed that there is another PR that suggested this:

#253.

However, it regards two dependencies, and does not modify setup.py.
Let's start with this one, first remove docopt, and later yarg.

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

Successfully merging this pull request may close these issues.

1 participant