Protocol 1.1: Upgrade protocol version (HAP-python 4.0.0)

[bdraco]

TODO:

• Implement timed writes Protocol 1.1: Add support for prepared writes (HAP-python 4.0.0) #366

[codecov]

Codecov Report

Merging #365 (11590c9) into dev (fddedae) will decrease coverage by 0.21%.
The diff coverage is 100.00%.

❗ Current head 11590c9 differs from pull request most recent head 078214d. Consider uploading reports for the commit 078214d to get more accurate results

@@            Coverage Diff             @@
##              dev     #365      +/-   ##
==========================================
- Coverage   91.28%   91.07%   -0.22%     
==========================================
  Files          20       20              
  Lines        2008     1949      -59     
  Branches      240      225      -15     
==========================================
- Hits         1833     1775      -58     
  Misses        131      131              
+ Partials       44       43       -1     

Impacted Files	Coverage Δ
pyhap/accessory_driver.py	90.46% <ø> (-0.58%)	⬇️
pyhap/accessory.py	96.62% <100.00%> (+0.24%)	⬆️
pyhap/const.py	100.00% <100.00%> (ø)
pyhap/hap_handler.py	81.23% <0.00%> (-0.57%)	⬇️
pyhap/characteristic.py	100.00% <0.00%> (ø)
pyhap/hap_protocol.py	100.00% <0.00%> (+0.62%)	⬆️

[bdraco]

Looks like we are almost there but we have to implement the /prepare endpoint and timed writes

[ikalchev]

Will wait for a final nod from you before merging this and releasing

[ikalchev]

Also I think some tests fail, mind if you take look if you do any final touches here?

[bdraco]

On it

…

On Jul 29, 2021, at 2:45 PM, Ivan Kalchev ***@***.***> wrote: Also I think some tests fail, mind if you take look if you do any final touches here? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#365 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFB7CCLYSDCTVVQ2HLPDOTT2GVWHANCNFSM5A7AQSOA>.

[ikalchev]

Thanks! I suspect sloppy merging on my part, so apologies.

[bdraco]

One of the PRs changed the behavior of the other. All fixed in #369

…

On Jul 29, 2021, at 2:47 PM, Ivan Kalchev ***@***.***> wrote: Thanks! I suspect sloppy merging on my part, so apologies. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#365 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFB7CDP364U624HBQFYKODT2GV4JANCNFSM5A7AQSOA>.

[bdraco]

Ideally what is merged now can be released as 3.5.3 or 3.6.0 and then merge this and release at 4.0.0 so we can do a bump in Home Assistant for 3.x and then in next month 4.x so the protocol version change and the other changes just merged get independent feedback

[bdraco]

rebased to make sure this is clean against dev