Skip to content
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

Use voluptuous for Honeywell #3298

Merged
merged 1 commit into from
Sep 11, 2016
Merged

Use voluptuous for Honeywell #3298

merged 1 commit into from
Sep 11, 2016

Conversation

fabaff
Copy link
Member

@fabaff fabaff commented Sep 9, 2016

Description:
Migration of the configuration check to voluptuous. Also this is includes an upgrade of somecomfort to 0.3.1 as discussed on gitter.

Related issue (if applicable): fixes 127528299

Example entry for configuration.yaml (if applicable):

climate:
  platform: honeywell
  username: YOUR_USERNAME
  password:  YOUR_PASSWORD
  region: REGION

Would be nice if somebody could take a look at the changes and run a quick test. Thanks.

@fabaff fabaff mentioned this pull request Sep 9, 2016
@balloob
Copy link
Member

balloob commented Sep 10, 2016

🐬

@fabaff fabaff merged commit 78313c7 into home-assistant:dev Sep 11, 2016
@fabaff fabaff deleted the voluptuous-honeywell branch September 11, 2016 07:21
@home-assistant home-assistant locked and limited conversation to collaborators Mar 17, 2017
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.

2 participants