Skip to content

Fixes linter warning in ZHA sensor#24406

Merged
dmulcahey merged 1 commit into
devfrom
frenck-2019-0064
Jun 8, 2019
Merged

Fixes linter warning in ZHA sensor#24406
dmulcahey merged 1 commit into
devfrom
frenck-2019-0064

Conversation

@frenck
Copy link
Copy Markdown
Member

@frenck frenck commented Jun 8, 2019

Description:

Fixed linter error introduced in #24373

Affects PR's (build failures because of this issue): #24399, #24400, #24402, #24403, #24405

Related issue (if applicable): n/a

Pull request with documentation for home-assistant.io (if applicable): n/a

Example entry for configuration.yaml (if applicable): n/a

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

If the code communicates with devices, web services, or third-party tools:

  • The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
  • New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
  • Untested files have been added to .coveragerc.

If the code does not interact with devices:

  • Tests have been added to verify that the new code works.

@ghost ghost assigned Adminiuga and dmulcahey Jun 8, 2019
@ghost
Copy link
Copy Markdown

ghost commented Jun 8, 2019

Hey there @dmulcahey, @Adminiuga, mind taking a look at this pull request as its been labeled with a integration (zha) you are listed as a codeowner for? Thanks!

This is a automatic comment generated by codeowners-mention to help ensure issues and pull requests are seen by the right people.

@dmulcahey
Copy link
Copy Markdown
Contributor

Sorry about that everyone and thanks for the assist @frenck

@dmulcahey dmulcahey merged commit 95d460c into dev Jun 8, 2019
@delete-merged-branch delete-merged-branch Bot deleted the frenck-2019-0064 branch June 8, 2019 11:29
@balloob balloob mentioned this pull request Jun 26, 2019
alandtse pushed a commit to alandtse/home-assistant that referenced this pull request Oct 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed integration: zha small-pr PRs with less than 30 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants