-
-
Notifications
You must be signed in to change notification settings - Fork 32k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix construct==2.8.17 Cause the plug-in failure #11089
Conversation
fix construct==2.8.17 Cause the plug-in failure
@@ -31,7 +31,7 @@ | |||
vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string, | |||
}) | |||
|
|||
REQUIREMENTS = ['python-miio==0.3.2'] | |||
REQUIREMENTS = ['python-miio==0.3.2','construct==2.8.16'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing whitespace after ','
Update requirements_all.txt
This is not going to be fixed by pinning the version on upstream; if pinning for the specific version in homeassistant is fine, then feel free to do that. I would prefer to have a clean solution (either by fixing python-miio to work with the new construct version, or by fixing construct if the problem lies there). |
I just tested the newer version out, and made mirobo to print out the version number. Seems to work fine here.
|
It doesn't matter, I just provide a way to the fastest possible repair, because time is limited, can't further wrong, just as a reminder, after all, a lot of people already can't use, thank you for your effort |
fix construct==2.8.17 Cause the plug-in failure
Description:
Related issue : fixes #11048 (comment)
At the same time update in python-miio
rytilahti/python-miio#140
See which side to submit first
See which side first submitted to solve the problem