Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

adding types annotations and py 2.7 support #176

Open
aogier opened this issue Jul 16, 2023 · 3 comments
Open

adding types annotations and py 2.7 support #176

aogier opened this issue Jul 16, 2023 · 3 comments

Comments

@aogier
Copy link

aogier commented Jul 16, 2023

dear maintainers, since I have recently started using this library a lot I would like to contribute a patch with types annotations so I stop putting comments for mypy :D

Seeing the code I find some parts with workarounds to support python 2.7 and although in setup.py I see a requirement >= 3.7 I am not sure if this compatibility is somehow a requirement of the project, which in this case would stop me from contributing on that.

What do you think? Are you interested in such a contribution ? I could also write a stub and put on the usual typeshed repo but I'd prefer to work on this upstream.

Thank you, regards

@scriptsrc
Copy link
Contributor

scriptsrc commented Jul 16, 2023 via email

@patricksanders
Copy link
Collaborator

This sounds great. I'm cool with dropping support for all EOL versions, which would bring us up to >= 3.8.

Thanks for your interest in contributing!

@aogier
Copy link
Author

aogier commented Jul 18, 2023

ok cool! I'll work on it, for now on my spare time so not very fast iterations but I hope to reach something working well enough soon then you will help me w/ what I'll miss out for sure and also my company will hopefully sponsor w/ my paid time sooner or later. Thank you! :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants