Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

Closed
wants to merge 6 commits into from
Closed

Update notify.lametric.markdown #4010

wants to merge 6 commits into from

Conversation

frittes
Copy link
Contributor

@frittes frittes commented Nov 18, 2017

Description:

Updated documentation for cycles config option and additional service data example.

Pull request in home-assistant (if applicable): home-assistant/core#10656

DubhAd and others added 6 commits November 17, 2017 10:34
* 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
Copy link
Member

@MartinHjelmare MartinHjelmare left a 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.

@frittes frittes changed the base branch from current to next November 18, 2017 20:16
@frittes
Copy link
Contributor Author

frittes commented Nov 18, 2017

Changed.

@MartinHjelmare
Copy link
Member

MartinHjelmare commented Nov 18, 2017

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.

@frittes
Copy link
Contributor Author

frittes commented Nov 18, 2017

@MartinHjelmare Need your further assistance. Created the PR here in git, not with a local repo.

What can I do to resolve the conflicts?

@MartinHjelmare
Copy link
Member

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.

@frittes
Copy link
Contributor Author

frittes commented Nov 19, 2017

@MartinHjelmare Created a new one.

#4017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants