-
Notifications
You must be signed in to change notification settings - Fork 100
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: Bump teslajsonpy to 3.12.2 #1067
Conversation
Please review other bumps. Other files will also also need to be changed. |
Updated the additional files. Not sure why the test would fail though as nothing there has been changed. |
Someone probably broke the test with a commit. |
Found reason for both tests that failed:
|
Thanks for figuring that out. Is the other PR #1069 a prereq of this? Normally I'd want to keep the test fix as a separate PR so the bump PR is very clean. |
Nope, no dependencies. PR #1069 was purely to update for HA 2024.11 and had nothing to do with teslajsonpy. :-) FYI, seems I screwed some things up so closing this PR for the bump and creating a new one instead that is clean. :-) |
@@ -5,11 +5,11 @@ | |||
|
|||
import logging | |||
|
|||
import voluptuous as vol |
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.
Did black do this?
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.
I would say yes, I didn't add it. :-)
Bump teslajsonpy to 3.12.2 which includes fix for API service