-
-
Notifications
You must be signed in to change notification settings - Fork 29k
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
miflora - fix for exception handling bug #12149
miflora - fix for exception handling bug #12149
Conversation
Why not use the pypi version? |
The version 0.2.0 of miflora on pypi does not yet include the bug fixes. And I do not have the permissions to create a new release. |
Have you proposed a new release in the miflora library? |
Then we should wait for a new release |
Linking requirements to GitHub is not allowed anymore. We will have to wait for a release. We're working on writing this down in an official policy |
Miflora version 0.3 is out. I'll update this PR on the weekend... |
Description:
With the release of miflora 0.2 there were two bugs in the exception handling. These are fixed in a development branch of miflora now. This PR pulls in this development branch and also handles the new exception type.
Once a the new release of miflora is out, we can switch back to the official releases: basnijholt/miflora#83
Related issue (if applicable): fixes #11815 #9048
Checklist:
If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
tox
run successfully. Your PR cannot be merged unless tests passREQUIREMENTS
variable (example).requirements_all.txt
by runningscript/gen_requirements_all.py
..coveragerc
.If the code does not interact with devices:
tox
run successfully. Your PR cannot be merged unless tests pass