-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Update notify.lametric.markdown #4010
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
* Rename configuration_yaml_by_aneisch to configuration_yaml_by_aneisch.markdown * Rename configuration_yaml_by_cy1701 to configuration_yaml_by_cy1701.markdown * Rename configuration_yaml_by_mf_social to configuration_yaml_by_mf_social.markdown * Update configuration_yaml_by_mf_social.markdown * Rename configuration_yaml_by_stanvx to configuration_yaml_by_stanvx.markdown
Tested and working with LC-60EQ10U
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I missed that you target the current
branch. Please instead target the next
branch. You can try by just pressing the edit button next to the PR name and switch target branch. If you get merge conflicts, you need to rebase from next
.
Changed. |
You need to rebase since your changes now contain other files that shouldn't be part of this PR. See https://home-assistant.io/developers/development_catching_up/ for general instructions. Replace branch names with corresponding branch names for this repo. |
@MartinHjelmare Need your further assistance. Created the PR here in git, not with a local repo. What can I do to resolve the conflicts? |
If you want to do this all online on github then I think you need to create a new PR, starting from the next branch. |
@MartinHjelmare Created a new one. |
Description:
Updated documentation for cycles config option and additional service data example.
Pull request in home-assistant (if applicable): home-assistant/core#10656