Switch to BooleanAttribute, req. Sopel 7.1+

sopel-irc / sopel-twitter

A Twitter plugin for Sopel

Other

2 stars 5 forks source link

Switch to BooleanAttribute, req. Sopel 7.1+ #27

Closed half-duplex closed 2 years ago

half-duplex commented 3 years ago

~Remove unnecessary futures~
~Remove coding~
Update Sopel version dep
Switch to BooleanAttribute, now that it works :wink:

dgw commented 3 years ago

Sopel 7.1 is not py3.6+ only. You're thinking of the future 8.0.

half-duplex commented 3 years ago

feh. I was thinking of the current development cycle of 8, where we're currently writing 3.6+ code, forgetting plugins aren't on the same cycle.

dgw commented 3 years ago

That, incidentally, is why sopel-irc/sopel#2164 will be included in the next 7.1.x maintenance release. No reason to force plugin updates just to avoid this single warning, when 8.0 is going to start warning about a lot more things.

dgw commented 2 years ago

I've punted this to the next minor version, since it feels icky to drop support for stuff in a patch version. We're doing the same with sopel-github: releasing the bumped Sopel requirement in a minor version (also 0.5.0, by coincidence).