Skip to content
This repository has been archived by the owner on Apr 23, 2022. It is now read-only.

change byte lengths to use Py_ssize_t #11

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

Conversation

jacobfeder
Copy link

This will solve some deprecation warning messages in more recent python versions.

@samuelcolvin
Copy link
Owner

LGTM, could you also update the readme.

github actions will also need to be added and travis removed before I can release a new version. Might not be until september.

@jacobfeder
Copy link
Author

Ok no problem. Thanks!

@jacobfeder
Copy link
Author

@samuelcolvin this deprecation warning is now an error in python 3.10 and prevents usage of the library - please let me know if there's anything I can do to to help get this change made. I'd be happy to take over maintenance if you'd like.

Thanks

@samuelcolvin
Copy link
Owner

samuelcolvin commented Dec 26, 2021 via email

@vadimszzz
Copy link

I'll try to fix it next week.

Thanks. Waiting for you.

@jacobfeder
Copy link
Author

I'll try to fix it next week.

On Sun, 26 Dec 2021, 20:40 Jacob Feder, @.> wrote: @samuelcolvin https://github.com/samuelcolvin this deprecation warning is now an error in python 3.10 and prevents usage of the library - please let me know if there's anything I can do to to help get this change made. I'd be happy to take over maintenance if you'd like. Thanks — Reply to this email directly, view it on GitHub <#11 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA62GGNOPCQOP7BQ4IGFICLUS54VHANCNFSM5BEVB2TQ . You are receiving this because you were mentioned.Message ID: @.>

Hi @samuelcolvin - bump on this. If you give repo access I'd also be happy to implement the travis/GH changes you had mentioned if you give me instructions on how you'd like it.

Best,
Jacob

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants