-
Notifications
You must be signed in to change notification settings - Fork 4
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
JOSS - rework setup.py #6
Comments
You should restrict the install to python 3 only since this is not python 2 compatible: |
Improve the setup as well such that there information in the PyPi webpage. |
I improved Is this file "nice enough"? Or do you see additional improvements? |
It seems almost good. Could you take care about the PEP8. |
It should work. PEP8 is in my to-do list, but I'll do the code styling all at once, I'm working on the coverage at present. |
The
setup.py
is not really nice. I would advice to use a template as hereThe text was updated successfully, but these errors were encountered: