search

nmoya / whatsapp-parser

Parser to the What's App log file.
MIT License
47 stars 23 forks source link

pip support: add setup.py and other relevant files to the project #15

Closed manu-chroma closed 7 years ago

manu-chroma commented 7 years ago

you can update the relevant fields in setup.py please test this to see if it works well for you.

you can install the project by running python setup.py install and then using wp_chat -h anywhere in the virtual env (if using any)

Let me know how it works for you. I'll make changes to the docs after you confirm :)

nmoya commented 7 years ago

@manu-chroma Ops, I got distracted by a meeting and didn't see your PR. I was about to open a new one: https://github.com/nmoya/whatsapp-parser/pull/16

Is it published in PyPi already? or that's the next step?

The only difference between our PRs is the author and author email.

nmoya commented 7 years ago

@manu-chroma The python setup.py install is just to test, right? The final goal is to do: pip install whatsapp-parser. and then wp_chat -h

is that what you had in mind?

manu-chroma commented 7 years ago

The final goal is to do: pip install whatsapp-parser. and then wp_chat -h

very true. that is just for testing purpose. there seems to be a little bug in setup.py I'll push a fix as soon as you merge either of the PR https://github.com/nmoya/whatsapp-parser/pull/15, https://github.com/nmoya/whatsapp-parser/pull/16

Is it published in PyPi already? or that's the next step?

that would be the next step. either one of us can do that

manu-chroma commented 7 years ago

Uploading to pypi shouldn't be a problem now reference: https://packaging.python.org/distributing/#uploading-your-project-to-pypi

nmoya commented 7 years ago

@manu-chroma I closed my PR. Let's merge #15

Please push the fix on setup.py

nmoya commented 7 years ago

@manu-chroma I'm starting the steps to upload to pypi.

manu-chroma commented 7 years ago

wait, I'll update this PR itself then. I'll amend the last commit.

nmoya commented 7 years ago

@manu-chroma Yes, I will only do the upload when the PR is merged. Just getting ready here on my side.

manu-chroma commented 7 years ago

Please review and merge. I've added your credentials and also fixed minor bug in file handling

nmoya commented 7 years ago

@manu-chroma Thanks for adding the credentials!

manu-chroma commented 7 years ago

@manu-chroma Yes, I will only do the upload when the PR is merged. Just getting ready here on my side.

sounds good to me! :smile: I'll stick around for sometime if you plan to upload it today!

nalbanders commented 7 years ago

I love getting these updates. You guys are awesome ;) Happy this is evolving

On Thu, Feb 2, 2017 at 10:48 AM, Manvendra Singh notifications@github.com wrote:

@manu-chroma https://github.com/manu-chroma Yes, I will only do the upload when the PR is merged. Just getting ready here on my side.

sounds good to me! 😄 I'll stick around for sometime if you plan to upload it today!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/nmoya/whatsapp-parser/pull/15#issuecomment-277046095, or mute the thread https://github.com/notifications/unsubscribe-auth/AG8Ct23CXI8kzim3jnU3uGSZ7OPYHIv2ks5rYiUAgaJpZM4L1YJ7 .

nmoya commented 7 years ago

@manu-chroma @nalbanders Thanks for all the help guys! 21 starts in this repository! Now published in python package manager: https://pypi.python.org/pypi/whatsapp-parser/1.0.0