-
Notifications
You must be signed in to change notification settings - Fork 8
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
feat: redesign library api #99
Conversation
Holy... You really want to be on each line of git blame 😆 |
Yeah, I might have went a bit overboard on some of the refactorings, but I felt some changes were necessary to improve overall quality and readability of the code. Also some minor features that were previously present are missing right now and not everything is 100% working and tested, so this will take a bit more work. |
Cool, I also get the same feeling with other's people code. But python isn't the language I use the moat, so I may have diverged from the python ways |
This PR introduces the following changes:
BleakConnection
andThermostat
homeassistant
dependency fromeq3btsmart
Thermostat