8000 Move HomeAssistantView to separate file. Convert http to async syntax. [skip ci] by fanthos · Pull Request #12982 · home-assistant/core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

< 8000 bdi class="js-issue-title markdown-title">Move HomeAssistantView to separate file. Convert http to async syntax. [skip ci] #12982

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

Merged
merged 5 commits into from
Mar 9, 2018

Conversation

fanthos
Copy link
Contributor
@fanthos fanthos commented Mar 8, 2018

Description:

Move HomeAssistantView to separate file.
Convert http(and http test) to async syntax.

Related issue (if applicable): fixes #

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):

http

Checklist:

  • The code change is tested and works locally.

If the code does not interact with devices:

  • Local tests with tox run successfully. Your PR cannot be merged unless tests pass
  • Tests have been added to verify that the new code works.

@fanthos fanthos changed the title Move HomeAssistantView to separate file. Convert http to async syntax. Move HomeAssistantView to separate file. Convert http to async syntax. [skip ci] Mar 8, 2018
@fanthos
Copy link
Contributor Author
fanthos commented Mar 8, 2018

pylint-dev/pylint#996
Found this

@balloob balloob merged commit 321eb2e into home-assistant:dev Mar 9, 2018
@balloob balloob mentioned this pull request Mar 9, 2018
@fanthos fanthos deleted the http_async branch April 12, 2018 07:11
@home-assistant home-assistant locked and limited conversation to collaborators Jul 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0