PythonistaGuild / TwitchIO

An Async Bot/API wrapper for Twitch made in Python.
https://twitchio.dev
MIT License
785 stars 163 forks source link

Remove dependency on typing_extensions #406

Closed joelsgp closed 1 year ago

joelsgp commented 1 year ago

Pull request summary

Literal is now in the standard library. Also remove the old annotations imports, as the relevant python versions are no longer supported.

Checklist

IAmTomahawkx commented 1 year ago

hi there, as the docs build has pointed out, 3.7 does not have Literal in the typing module. as such, the typing_extensions version is used throughout the module. Additionally, you appear to have removed the future annotation imports?

IAmTomahawkx commented 1 year ago

If you have further PRs to make, i would suggest joining our discord, which is linked in the README, to discuss them first

joelsgp commented 1 year ago

3.7 is about to go end-of-life https://devguide.python.org/versions/

I'm already in the discord but I should have checked the docs section on this before submitting. I was mistaken on the future imports, I thought that was already mandatory but it's tbd https://docs.python.org/3/library/__future__.html

chillymosh commented 1 year ago

Python 2.7 is also EoL but people still use it and rely on it in companies :)

joelsgp commented 1 year ago

Python 2.7 is also EoL but people still use it and rely on it in companies :)

I'm sorry to hear that, I will pray for them 🙏

IAmTomahawkx commented 1 year ago

our support is not based around the timing the python org provides for lifecycles. We will provide support for 3.7+ for the lifetime of this semver cycle (V2). We will drop support for older python versions in our next cycle (V3)

joelsgp commented 1 year ago

Understood, and I'd appreciate it if the semantic versioning / version branch model was explained in https://github.com/TwitchIO/TwitchIO#contributing for future reference