Closed emontnemery closed 6 months ago
The breaking change sentence sounds weird. It sounds like the callback is called when a request has changed signature.
I think you should document what the callback parameters mean, at least in the PR description and maybe in the readme.
Breaking change
The signature of callbacks called when requests are completed has been changed:
New callback signature:
Change
Callback functions are now called not only on success but also on error. This prevents failing requests from hanging forever. Also, many user facing functions which send messages to the remote device but do not accept a callback now accept a timeout parameter.
Background in PR https://github.com/home-assistant-libs/pychromecast/pull/779
Needs https://github.com/home-assistant-libs/pychromecast/pull/784