Skip to content

Conversation

@JonnoFTW
Copy link

@JonnoFTW JonnoFTW commented May 8, 2015

Added a setup.py to make this package easier to use. Requires setuptools.

@MojoJolo
Copy link
Contributor

@JonnoFTW sorry I just saw this pull request. I wonder why it didn't alert me in my email. Will try this out and merge if all is good.

@neurosnap
Copy link

https://pypi.python.org/pypi/textteaser/0.3

I already see "textteaser" python package, if you want to upload this package to pypi -- which is recommended -- then you'll prolly want to go with a different name.

@MojoJolo
Copy link
Contributor

Thanks. It looks like he's using a previous version of TextTeaser that doesn't work today. I wonder if it is possible to replace it.

@neurosnap
Copy link

I would recommend something like 'textteaserpy' or even 'txtteaser' if you want to change the name for pypi.

I would also recommend manually writing the install_requires since there is only one dependency and reading the file line-by-line has its complications: empty lines, invalid package names, and also doesn't allow the ability to download packages from github.

See this http://stackoverflow.com/a/16624700/1713216 post for more information on reading the requirements.txt for setup.py

@JonnoFTW
Copy link
Author

For all this to happen, the DataTeaser team would need to come up with an appropriate package name, (I just wrote the setup.py file).

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.

3 participants