Skip to content
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

Bumb requirement version for dacite GREATER than 1.8.0 #132

Closed

Conversation

sdrapha
Copy link

@sdrapha sdrapha commented Nov 10, 2023

Similar PR to the other one already posted here. only difference though is, since I started to troubleshoot my problem with the Roborock integration, there was already a newer version of Dacite, the 1.8.1.
HOWEVER, I did tried to explicitly ask for the newer version, which was also giving me some problems with roborock for some weird reason.
So the actual fix now, is to ask on the requirements for a version GREATER than 1.8.0 >=1.8.0 instead of pinning any specific version ==. And letting the system deal with. I actually get the 1.8.1 installed anyway when I check with pip show dacite.
This way now, I have everything working, Govee and Roborock, everytime, surviving reboots.

@LaggAt
Copy link
Owner

LaggAt commented Nov 12, 2023

Thanks, thats a duplicate to a merged reqeust.
https://github.com/LaggAt/hacs-govee/pull/124/files

@LaggAt LaggAt closed this Nov 12, 2023
@dcnoren
Copy link

dcnoren commented Dec 6, 2023

Thanks, thats a duplicate to a merged reqeust. https://github.com/LaggAt/hacs-govee/pull/124/files

Hi @LaggAt , I think this PR is different. @sdrapha is asking to make the requirement 1.8.0 OR HIGHER. The PR you put in 124 is saying requires exactly 1.8.0.

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

Successfully merging this pull request may close these issues.

3 participants