Skip to content
This repository has been archived by the owner on Oct 30, 2019. It is now read-only.

Minor typo and clarification. #135

Closed
wants to merge 1 commit into from
Closed

Minor typo and clarification. #135

wants to merge 1 commit into from

Conversation

Landrash
Copy link
Collaborator

@Landrash Landrash commented Mar 9, 2018

Description:

Minor typo fix and clarification of how to run command if run without sudo.

Checklist:

  • The code change is tested and works locally.
  • Script has validation check of the job.

If pertinent:

  • Created/Updated documentation at /docs

@Landrash Landrash requested a review from ludeeus March 9, 2018 20:45
ludeeus
ludeeus approved these changes Mar 10, 2018
View reviewed changes
Copy link
Member

@ludeeus ludeeus left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great 👍
But there seems to be an conflict.

@home-assistant home-assistant deleted a comment from stickler-ci Mar 10, 2018
@ludeeus ludeeus added this to the v0.8.0 milestone Mar 12, 2018
@ludeeus
Copy link
Member

ludeeus commented Mar 12, 2018

Some of this is actually fixed in current dev.

@Landrash
Copy link
Collaborator Author

Closing since it's fixed in later pr's.

@Landrash Landrash closed this Mar 13, 2018
@ludeeus ludeeus mentioned this pull request Mar 23, 2018
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ludeeus ludeeus ludeeus approved these changes

Assignees
No one assigned
Labels
cla-signed enhancement
Projects
None yet
Milestone
v0.8.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Landrash @ludeeus @homeassistant