Skip to content

Conversation

balloob
Copy link
Member

@balloob balloob commented May 2, 2016

Description:
Some clean up related to scripts. Also alphabetically sorted CONF constants in const file.

Related issue (if applicable): #

Example entry for configuration.yaml (if applicable):

Checklist:

If code communicates with devices:

  • Local tests with tox run successfully. Your PR cannot be merged unless tests pass
  • New dependencies have been added to the REQUIREMENTS variable (example).
  • New dependencies are only imported inside functions that use them (example).
  • New 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:

  • 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.

@balloob balloob merged commit 79653a6 into dev May 3, 2016
@balloob balloob deleted the random-fixes branch May 3, 2016 05:05
@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.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant