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

Honor PEP8 naming convention #8909

Merged
merged 2 commits into from
Aug 10, 2017
Merged

Honor PEP8 naming convention #8909

merged 2 commits into from
Aug 10, 2017

Conversation

fabaff
Copy link
Member

@fabaff fabaff commented Aug 10, 2017

Description:

  • Honor PEP8 naming convention
  • Remove redundant parentheses
  • Align quoting

Example entry for configuration.yaml (if applicable):

updater:

Checklist:

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.

@fabaff fabaff merged commit e3236d1 into dev Aug 10, 2017
@fabaff fabaff deleted the pep8-updater branch August 10, 2017 17:31
@fabaff fabaff mentioned this pull request Aug 12, 2017
dethpickle pushed a commit to dethpickle/home-assistant that referenced this pull request Aug 18, 2017
* Honor PEP8 naming convention

* Update validator
@home-assistant home-assistant locked and limited conversation to collaborators Dec 11, 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.

4 participants