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

remove argparse from requirements.txt #215

Open
blshkv opened this issue Feb 13, 2020 · 0 comments
Open

remove argparse from requirements.txt #215

blshkv opened this issue Feb 13, 2020 · 0 comments

Comments

@blshkv
Copy link

blshkv commented Feb 13, 2020

https://github.com/aboul3la/Sublist3r/blob/master/requirements.txt#L1

I think this line pulls the package called "argparse' unnecessarily. According to the official page,
https://pypi.org/project/argparse/ it has been unmaintained for quite some time now and was developed for old python compatibility only.

The PyPi argparse package is mostly for people who want to have argparse on
older Pythons, like < 2.7 or < 3.2 because it was not in stdlib back then.

python 2.7 and >3.4 supports "import argparse" natively using build-in functionality:
https://docs.python.org/3/howto/argparse.html

So you can simply drop that requirement.

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

1 participant