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.

Sign up for GitHub

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

Jump to bottom

Use voluptuous for Z-Wave #3679

Merged
merged 1 commit into from
Oct 5, 2016
Merged

Use voluptuous for Z-Wave #3679

merged 1 commit into from
Oct 5, 2016

Conversation

fabaff
Copy link
Member

@fabaff fabaff commented Oct 3, 2016

Description:
Migration of the configuration check to voluptuous.

Related issue (if applicable): fixes 127528299

Example entry for configuration.yaml (if applicable):

zwave:
  usb_path: /dev/ttyUSB0
  config_path: /usr/local/share/python-openzwave/config
  polling_interval: 60000
  customize:
    sensor.greenwave_powernode_6_port_energy_10:
        polling_intensity: 1

@fabaff fabaff mentioned this pull request Oct 3, 2016
Closed
@balloob
Copy link
Member

balloob commented Oct 5, 2016

Can anyone running Z-Wave test this? @turbokongen @robbiet480

turbokongen
turbokongen approved these changes Oct 5, 2016
View reviewed changes
Copy link
Contributor

@turbokongen turbokongen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and working 👍

@turbokongen turbokongen merged commit 0c68f38 into home-assistant:dev Oct 5, 2016
@fabaff fabaff deleted the voluptuous-zwave branch October 5, 2016 14:27
hartmms pushed a commit to hartmms/home-assistant that referenced this pull request Oct 5, 2016
@fabaff @hartmms
Migrate to voluptuous (home-assistant#3679)
336d444
@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.
Reviewers

@turbokongen turbokongen turbokongen approved these changes

Assignees
No one assigned
Labels
Hacktoberfest voluptuous
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@fabaff @balloob @turbokongen @robbiet480