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

Remove spaces from Xiami switch attributes #8952

Merged
merged 2 commits into from
Aug 13, 2017

Conversation

syssi
Copy link
Member

@syssi syssi commented Aug 13, 2017

Fixes #8808.

@mention-bot
Copy link

@syssi, thanks for your PR! By analyzing the history of the files in this pull request, we identified @Danielhiversen to be a potential reviewer.

ATTR_POWER_CONSUMED = 'Power consumed'
ATTR_IN_USE = 'In use'
ATTR_LOAD_POWER = 'load_power' # Load power in watts (W)
ATTR_POWER_CONSUMED = 'power_consumed' # Total (lifetime) power consumption in watts

Choose a reason for hiding this comment

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

at least two spaces before inline comment
line too long (84 > 79 characters)

ATTR_LOAD_POWER = 'Load power' # Load power in watts (W)
ATTR_POWER_CONSUMED = 'Power consumed'
ATTR_IN_USE = 'In use'
ATTR_LOAD_POWER = 'load_power' # Load power in watts (W)

Choose a reason for hiding this comment

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

at least two spaces before inline comment

@balloob balloob changed the title Attributes of the xiaomi zigbee plug changed. Remove spaces from Xiami switch attributes Aug 13, 2017
@Danielhiversen Danielhiversen merged commit 73d6227 into home-assistant:dev Aug 13, 2017
dethpickle pushed a commit to dethpickle/home-assistant that referenced this pull request Aug 18, 2017
* Attributes of the xiaomi zigbee plug changed.

* Reformat.
@balloob balloob mentioned this pull request Aug 25, 2017
@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.

7 participants