core
core copied to clipboard
Add broadlink climate platform
Breaking change
Proposed change
Added climate platform for broadlink devices.
Type of change
- [ ] Dependency upgrade
- [ ] Bugfix (non-breaking change which fixes an issue)
- [ ] New integration (thank you!)
- [x] New feature (which adds functionality to an existing integration)
- [ ] Breaking change (fix/feature causing existing functionality to break)
- [ ] Code quality improvements to existing code or addition of tests
Additional information
- This PR fixes or closes issue: fixes #
- This PR is related to issue:
- Link to documentation pull request: https://github.com/home-assistant/home-assistant.io/pull/15762
Checklist
- [x] The code change is tested and works locally.
- [x] Local tests pass. Your PR cannot be merged unless tests pass
- [x] There is no commented out code in this PR.
- [x] I have followed the development checklist
- [x] The code has been formatted using Black (
black --fast homeassistant tests) - [ ] Tests have been added to verify that the new code works.
If user exposed functionality or configuration variables are added/changed:
- [ ] Documentation added/updated for www.home-assistant.io
If the code communicates with devices, web services, or third-party tools:
- [x] The manifest file has all fields filled out correctly.
Updated and included derived files by running:python3 -m script.hassfest. - [ ] New or updated dependencies have been added to
requirements_all.txt.
Updated by runningpython3 -m script.gen_requirements_all. - [ ] For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description.
- [x] Untested files have been added to
.coveragerc.
The integration reached or maintains the following Integration Quality Scale:
- [ ] No score or internal
- [ ] 🥈 Silver
- [ ] 🥇 Gold
- [ ] 🏆 Platinum
To help with the load of incoming pull requests:
- [ ] I have reviewed two other open pull requests in this repository.
Hey there @danielhiversen, @felipediel, @l-i-am, mind taking a look at this pull request as it has been labeled with an integration (broadlink) you are listed as a code owner for? Thanks!
(message by CodeOwnersMention)
I think it's now the fifth try or something like that :D https://github.com/home-assistant/core/pull/45780
Even if you only apply this to climate.py and not to the other platforms, I still think this would benefit from being migrated to attr format: https://developers.home-assistant.io/docs/core/entity/#entity-class-or-instance-attributes
You can see an example in #61605
Even if you only apply this to
climate.pyand not to the other platforms, I still think this would benefit from being migrated to attr format: https://developers.home-assistant.io/docs/core/entity/#entity-class-or-instance-attributesYou can see an example in #61605
After 2 years of trying to get it in, I jsut would be glad if it was finally added :D I don't have that much time at my disposal.
After 2 years of trying to get it in, I just would be glad if it was finally added :D I don't have that much time at my disposal.
Agree, let's get it in and then optimize.
Simone
There is a small conflict on the CODEOWNERS file. The format has changed slightly to now include the tests folder.
Could you rebase and adjust accordingly?
There is a small conflict on the CODEOWNERS file. The format has changed slightly to now include the
testsfolder. Could you rebase and adjust accordingly?
Can you please tell me exactly how to rebase correctly? Everytime I did it in the past everything went straight to hell.
Can you please tell me exactly how to rebase correctly? Everytime I did it in the past everything went straight to hell.
Did you try following https://developers.home-assistant.io/docs/development_catching_up ?
Simone
I followed that guide, rebase was successful, but unable to push, -f and now I have the current situation ^^
Hi there 👋
This PR seems to be stuck a bit. Can we get it unstuck? @majuss Could you rebase the PR onto the latest dev (and fix the conflicts during the rebase?). Would be awesome 👍
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.
I will work on this PR in the next weeks.
@majuss do you want some assistance with this PR? e.g. cleaning up the merge conflicts. I am using your custom_component and would be glad to help make this part of core.
This PR seems stale so setting as draft.