Client.write_data - remove potentially dangerous default

[briantist]

Fixes #1104

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (27dad5b) 87.12% compared to head (eeab693) 87.38%.
Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1120      +/-   ##
==========================================
+ Coverage   87.12%   87.38%   +0.25%     
==========================================
  Files          64       66       +2     
  Lines        3162     3227      +65     
==========================================
+ Hits         2755     2820      +65     
  Misses        407      407              

Files	Coverage Δ
hvac/v1/__init__.py	90.00% <ø> (ø)

... and 3 files with indirect coverage changes

[tot19]

Looks good to me ... Good catch. One potential addition here would be to add a test to catch future instances of this? Know you mentioned pylint, and I generally have had good experience with it.

[briantist]

Thanks for the review @tot19 ! I think I will address detection separately. Before adding a fourth(!) linter/checker to the project, I will probably evaluate this open PR #1059 to add ruff. Maybe it would have caught this too, and we can reduce the number of tools. In the meantime I'll merge this one.