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

Fix spelling error in log output #17963

Merged
merged 1 commit into from Oct 29, 2018

Conversation

Projects
None yet
6 participants
@bitglue
Member

bitglue commented Oct 29, 2018

I noticed a spelling error in log output when I upgraded. I fixed it.

@houndci-bot

Some files could not be reviewed due to errors:

Traceback (most recent call last):
Traceback (most recent call last):
  File "/home/linters/.local/bin/flake8", line 7, in 
    from flake8.main.cli import main
ModuleNotFoundError: No module named 'flake8'
@cgarwood

This comment has been minimized.

Member

cgarwood commented Oct 29, 2018

Looks good, but please fill out the PR template.

@bitglue

This comment has been minimized.

Member

bitglue commented Oct 29, 2018

Fill it out with what? There's not a single applicable item on it for such a trivial change.

@cgarwood

This comment has been minimized.

Member

cgarwood commented Oct 29, 2018

At minimum, a short description should be provided/left in since we expect that from other contributors, and since there's been a lot of spam Hacktoberfest PRs getting submitted with blank blank PR information.

@fabaff

fabaff approved these changes Oct 29, 2018

Thanks 🐦

@balloob balloob merged commit af5eacf into home-assistant:dev Oct 29, 2018

3 of 4 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Hound No violations found. Woof!
WIP Legacy commit status override — see details
Details
cla-bot Everyone involved has signed the CLA

@wafflebot wafflebot bot removed the in progress label Oct 29, 2018

@bitglue bitglue deleted the bitglue:migration_spelling branch Oct 30, 2018

@balloob balloob referenced this pull request Nov 9, 2018

Merged

0.82 #18335

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment