Skip to content

Remove some unused imports#15529

Merged
balloob merged 1 commit intohome-assistant:devfrom
scop:unused-imports
Jul 18, 2018
Merged

Remove some unused imports#15529
balloob merged 1 commit intohome-assistant:devfrom
scop:unused-imports

Conversation

@scop
Copy link
Copy Markdown
Member

@scop scop commented Jul 18, 2018

Description:

Remove some unused imports.

Related issue (if applicable): Some discussion in #15487

Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>

Example entry for configuration.yaml (if applicable):

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass

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

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

  • New dependencies have been added to the REQUIREMENTS variable (example).
  • New dependencies are only imported inside functions that use them (example).
  • New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
  • New files were added to .coveragerc.

If the code does not interact with devices:

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

@scop scop requested a review from a team as a code owner July 18, 2018 07:33
@homeassistant homeassistant added cla-signed core small-pr PRs with less than 30 lines. labels Jul 18, 2018
@ghost ghost added the in progress label Jul 18, 2018
@balloob balloob merged commit 2781796 into home-assistant:dev Jul 18, 2018
@ghost ghost removed the in progress label Jul 18, 2018
@scop scop deleted the unused-imports branch July 18, 2018 08:57
@balloob balloob mentioned this pull request Aug 3, 2018
@home-assistant home-assistant locked and limited conversation to collaborators Dec 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed core small-pr PRs with less than 30 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants