Skip to content

Conversation

@CoMPaTech
Copy link
Member

Fix m_adam_heating (through reviewing core pr)

@CoMPaTech CoMPaTech added the bug Something isn't working label May 3, 2023
@CoMPaTech CoMPaTech requested a review from bouwew as a code owner May 3, 2023 21:43
@CoMPaTech CoMPaTech self-assigned this May 3, 2023
@codecov
Copy link

codecov bot commented May 3, 2023

Codecov Report

Merging #309 (72d7927) into main (6778000) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #309   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            6         6           
  Lines         2629      2629           
=========================================
  Hits          2629      2629           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@bouwew bouwew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change is correct.
But, the name domestic_hot_water_setpoint has changed for all fixtures to max_dhw_temperature as a part of the v0.31.3 updates.

Copy link
Contributor

@bouwew bouwew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Named fixed.

@bouwew bouwew merged commit a7594e6 into main May 4, 2023
@bouwew bouwew deleted the fixture_fix branch May 4, 2023 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants