search

SecKatie / wyzeapy

35 stars 26 forks source link

Remove cchardet #67

Closed brg468 closed 1 year ago

brg468 commented 1 year ago

As discussed here https://github.com/JoshuaMulliken/wyzeapy/issues/66. and here https://github.com/JoshuaMulliken/ha-wyzeapi/issues/462

HA is blocking cchardet in release 2023.3 as it's unmaintained and doesn't support Python 3.11. It doesn't appear that its used in the code so this removes it as a dependency to fix the conflict.

JoeSchubert commented 1 year ago

LGTM. This should have no effects on the actual package other than removing cchardet as a listed dependency, which should fix the Home assistant blockage.

debsahu commented 1 year ago

I'm on HA 2023.3.0 and this might fix the broken add on in this version of HA. Could someone merge this after testing?

SecKatie commented 1 year ago

I will get a version pushed once I am back in front of the computer