-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Merge documentation update for PR 4981 #1652
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@Duoxilian, thanks for your PR! By analyzing the history of the files in this pull request, we identified @fabaff, @skyval and @Danielhiversen to be potential reviewers. |
I am not sure whether I did this right, because I don't see my changes in the list above. For what it is worth, I updated components.climate on fork duoxilian, 12389ec. |
Yeah, something is not right with this branch. I can try to pull your commit and push it upstream. I'll see what I can do. Your changes look good. |
@Duoxilian there's a tutorial how to do the PR here: Only difference is that you should change branch to It looks as your existing commit is orphaned on github, cause I can't find any way of fetching it to my local repo. |
Thanks. The trick was going to next first. |
Description:
Pull request in home-assistant (if applicable): home-assistant/home-assistant#