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

Made repo pip installable #19

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

Conversation

raunakdoesdev
Copy link

As Rdkit is now available on PYPI, I made this project pip installable and uploaded it on PYPI myself:
https://pypi.org/project/descriptastorus/

If you would like to merge this PR and upload the project to PYPI on your own account, I will happily take down my PYPI project so you can do so.

@bp-kelley
Copy link
Owner

I appreciate you doing this, I'm leary of taking out the pandas-flavor though, this package is used by py-janitor which is why it was used in the first place. Is this not required anymore with pandas?

@bp-kelley
Copy link
Owner

@sauhaardac there is a pretty big bug in the descriptastorus on pipy, I'd like to add pushing to pypi as part of my continuous integration. Is it possible to turn control over the pypi repository to me so I can always push new releases?

@raunakdoesdev
Copy link
Author

raunakdoesdev commented Oct 13, 2022

@bp-kelley Happy to do so! I need your pypi username to be able to do that, could you send that over? If not willing to share here, you can also contact me at "[email protected]"

@bp-kelley
Copy link
Owner

bp-kelley commented Oct 18, 2022 via email

@raunakdoesdev
Copy link
Author

raunakdoesdev commented Oct 18, 2022 via email

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