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.

Sign up for GitHub

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

Jump to bottom

Update setup.py #22

Open
cboddy opened this issue Sep 2, 2019 · 3 comments
Open

Update setup.py #22

cboddy opened this issue Sep 2, 2019 · 3 comments
Assignees
@soniapignorel

Comments

@cboddy
Copy link
Collaborator

cboddy commented Sep 2, 2019 •
edited
Loading

the long description should point to README.md

https://github.com/manahl/feedback_tool/blob/github/setup.py#L11
readme.rst doesn't exist

Author should be Man Alpha Tech ?

https://github.com/manahl/feedback_tool/blob/github/setup.py#L28

download URL is hard-coded to v1.0.0

https://github.com/manahl/feedback_tool/blob/github/setup.py#L25
Can we add this to pypi instead of hosting on github?
@cboddy
Copy link
Collaborator Author

cboddy commented Sep 2, 2019

Also it should have a short description.

@cboddy
Copy link
Collaborator Author

cboddy commented Sep 2, 2019

suggestion: reconcile the fields and values against okcli's setup.py or PyBloqs'.

@soniapignorel
Copy link
Contributor

Now in PyPI: https://pypi.org/project/feedback-tool/1.0.0/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@soniapignorel soniapignorel

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cboddy @soniapignorel