Skip to content
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

have a comment on the format of the repo #7366

Merged
merged 2 commits into from
Nov 11, 2018

Conversation

maxandersen
Copy link
Contributor

i had to read the source of run.sh to figure out the logic - thought it would
be good to have a comment about the format.

Description:

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

Checklist:

  • Branch: next is for changes and new documentation that will go public with the next home-assistant release. Fixes, changes and adjustments for the current release should be created against current.
  • The documentation follows the standards.

i had to read the source of run.sh to figure out the logic - thought it would
be good to have a comment about the format.
@ghost ghost added the to-do label Nov 4, 2018
@ghost ghost assigned frenck Nov 11, 2018
@ghost ghost added ready-for-review This PR needs to be reviewed and removed to-do labels Nov 11, 2018
@frenck frenck added enhancement current This PR goes into the current branch labels Nov 11, 2018
@frenck
Copy link
Member

frenck commented Nov 11, 2018

Thanks, @maxandersen!

I've made some tiny tweaks. Can be merged as soon as the build passes.

@frenck frenck merged commit ae6a947 into home-assistant:current Nov 11, 2018
@ghost ghost removed the ready-for-review This PR needs to be reviewed label Nov 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
current This PR goes into the current branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants