Broadlink IR | Home Assistant | Page 1

tribal lodge Apr 21, 2026, 1:58 PM

#

rotund crater Apr 21, 2026, 1:59 PM

#

shadow shale Apr 21, 2026, 2:00 PM

#

cc @subtle turret

waxen schooner Apr 21, 2026, 2:07 PM

#

I'd agree with a revert for now, it just seems to violate too many of our principles

rotund crater Apr 21, 2026, 2:12 PM

#

Yes, let's do it.

#

(The functionality as such is of course very welcome)

subtle turret Apr 21, 2026, 2:20 PM

#

No problem. I will make a new PR soon and hope it will be better 😅
Sorry for the mess. First time...

shadow shale Apr 21, 2026, 2:20 PM

#

No worries

rotund crater Apr 21, 2026, 2:21 PM

#

Yeah, I didn't mean to put any blame on you @subtle turret. Adding IR support to the broadlink integration is for sure wanted.

#

It's just that our review process didn't work this time

tribal lodge Apr 21, 2026, 4:15 PM

#

Should we have the label 'revert' be accepted by the required-labels bot? None of the available 'required lables' make sense in case of a revert IMO

shadow shale Apr 21, 2026, 4:15 PM

#

Code quality I guess

#

A revert can also be a bugfix or breaking change

tribal lodge Apr 21, 2026, 4:16 PM

#

hmm, I guess so

feral saffron Apr 21, 2026, 11:39 PM

#

Sorry about that

#Broadlink IR