Fix compatibility with zeroconf 0.27#263
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #263 +/- ##
=======================================
Coverage 64.45% 64.45%
=======================================
Files 16 16
Lines 1761 1761
Branches 189 189
=======================================
Hits 1135 1135
Misses 573 573
Partials 53 53
|
|
Should we also update requirements.txt to require zaeroconf >=0.27? |
elupus
left a comment
There was a problem hiding this comment.
The array returned from getaddrinfo should probably have been used.
|
Then again, the lib doesn't do that for it's tests. So.. |
It should work with both newer and older versions due to the keyword args being added. Sadly we are going to need another release sometime soon to unblock Home Assistant from upgrading. |
|
Verified with busy home that both zeroconf 0.26.3 and 0.27.0 work with the change |
|
Will release later today |
Thanks. 2.9.0 has been going great, no more cpu spikes or unexpected reconnects after running it since you published. |
|
2.9.1 is now GA |
Fixes #262