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

Make elodie pip installable #345

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

Make elodie pip installable #345

wants to merge 3 commits into from

Conversation

waynew
Copy link

@waynew waynew commented Dec 3, 2019

I have a sample setup.py file which should make Elodie pip-installable.

I also went ahead and registered the project on pypi so it didn't get picked up in the interim. @jmathai let me know what your pypi username is and I'll go ahead and make you an owner.

I moved elodie.py into the project folder so it could be used as an entrypoint. You should be able to check it out by (currently) running python -m pip install --user --pre elodie. The pre flag is only necessary because I uploaded the version as 0.0.1b.

It's pretty easy to build and release to pypi, I just used:

python setup.py bdist_wheel
twine check dist/elodie-0.0.1b-py3-none-any.whl 
twine upload dist/elodie-0.0.1b-py3-none-any.whl

Let me know if you have any questions!
HTH


I also went ahead and updated the gitignore to ignore vim swap files, and a couple of other potential folders. I also added a pre-commit config that might be useful for replacing the existing .pre-commit file. I didn't get it running the nosetest, but it did fire on flake8 (and there were a ton of violations in elodie.py that I didn't fix because that was a bit out of scope for this).

- Ignore vim swap files
- Ignore egg directories
@waynew
Copy link
Author

waynew commented Dec 3, 2019

Closes #41

@CLAassistant
Copy link

CLA assistant check
All committers have signed the CLA.

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.

2 participants