Nibe S1156

[hajekmi]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.15%. Comparing base (34541b6) to head (2a41278).
Report is 3 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #198      +/-   ##
==========================================
+ Coverage   70.13%   70.15%   +0.02%     
==========================================
  Files          14       14              
  Lines        1356     1357       +1     
==========================================
+ Hits          951      952       +1     
  Misses        405      405              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[yozik04]

I am thinking if it is similar to S1256? At the moment we have S1155 and S1255 merged into one file: s1155_s1255. I think we need to do the same for S1156 and S1256.

[yozik04]

With this PR current S1256 HA setups will have less registers. It used s1155_s1255 before and new separate CSV has less registers defined.

[hajekmi]

I agree that S1156 and S1256 will have similar registers. The only difference is the integrated tray, but the temperature sensors will be the same.
Thanks