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

Change docopt to the maitained one #37

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

miso-belica
Copy link
Member

I found that the docopt of this repo was no longer maintained and cause some issues here
lss233/chatgpt-mirai-qq-bot#62 (comment).

Details can be found here
docopt/docopt#502.

Therefore, I suggest the use of https://github.com/jazzband/docopt-ng to replace docopt.

@miso-belica
Copy link
Member Author

@craigmaloney can you merge and publish this please? I think I don't have permissions to PyPi.

@craigmaloney
Copy link
Contributor

craigmaloney commented Feb 21, 2023

@craigmaloney can you merge and publish this please? I think I don't have permissions to PyPi.

Sadly I don't have permission to push this to PyPI either. It might be time to fork breadability into its own project, but I don't have the mental bandwidth to maintain it. Consider this my permission to do so (as much as my permission on this matters)

@miso-belica
Copy link
Member Author

Sorry, I should check before. It is public info in PyPI. @mitechie I see you are a maintainer in PyPI. Do you have time to publish this?

@miso-belica
Copy link
Member Author

Also, I see CI is not working anymore probably because TravisCI migrated to new domain and cancelled the old service. I could create a Github action to run tests in PRs and publish code to PyPI. I have one in https://github.com/miso-belica/sumy/blob/main/.github/workflows/publish-to-pypi.yml so it should be quite easy.

@miso-belica
Copy link
Member Author

@mitechie sorry to bother you. I know you might be just busy but I hope you are fine. You can add me as a maintainer in PyPI and I can do this myself. But I understand if you don't want to. It is just a suggestion. But please respond so we know you saw it and you are fine 🙂

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