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 globally disabled pylint issues #8005

Merged
merged 1 commit into from Jun 13, 2017
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 0 additions & 1 deletion homeassistant/components/calendar/google.py
Expand Up @@ -39,7 +39,6 @@ def setup_platform(hass, config, add_devices, disc_info=None):
for data in disc_info[CONF_ENTITIES] if data[CONF_TRACK]])


# pylint: disable=too-many-instance-attributes
class GoogleCalendarEventDevice(CalendarEventDevice):
"""A calendar event device."""

Expand Down
2 changes: 1 addition & 1 deletion homeassistant/components/climate/wink.py
Expand Up @@ -45,7 +45,7 @@ def setup_platform(hass, config, add_devices, discovery_info=None):
add_devices([WinkAC(climate, hass, temp_unit)])


# pylint: disable=abstract-method,too-many-public-methods, too-many-branches
# pylint: disable=abstract-method
class WinkThermostat(WinkDevice, ClimateDevice):
"""Representation of a Wink thermostat."""

Expand Down
1 change: 0 additions & 1 deletion homeassistant/components/device_tracker/owntracks.py
Expand Up @@ -116,7 +116,6 @@ def decrypt_payload(topic, ciphertext):
"key for topic %s", topic)
return None

# pylint: disable=too-many-return-statements
def validate_payload(topic, payload, data_type):
"""Validate the OwnTracks payload."""
try:
Expand Down
1 change: 0 additions & 1 deletion homeassistant/components/media_player/aquostv.py
Expand Up @@ -108,7 +108,6 @@ def wrapper(obj, *args, **kwargs):
class SharpAquosTVDevice(MediaPlayerDevice):
"""Representation of a Aquos TV."""

# pylint: disable=too-many-public-methods
def __init__(self, name, remote, power_on_enabled=False):
"""Initialize the aquos device."""
global SUPPORT_SHARPTV
Expand Down
4 changes: 0 additions & 4 deletions homeassistant/components/media_player/plex.py
Expand Up @@ -134,7 +134,6 @@ def setup_plexserver(host, token, hass, config, add_devices_callback):
track_utc_time_change(hass, lambda now: update_devices(), second=30)

@util.Throttle(MIN_TIME_BETWEEN_SCANS, MIN_TIME_BETWEEN_FORCED_SCANS)
# pylint: disable=too-many-branches
def update_devices():
"""Update the devices objects."""
try:
Expand Down Expand Up @@ -231,11 +230,9 @@ def plex_configuration_callback(data):
}])


# pylint: disable=too-many-instance-attributes, too-many-public-methods
class PlexClient(MediaPlayerDevice):
"""Representation of a Plex device."""

# pylint: disable=too-many-arguments
def __init__(self, config, device, session, plex_sessions,
update_devices, update_sessions):
"""Initialize the Plex device."""
Expand Down Expand Up @@ -299,7 +296,6 @@ def __init__(self, config, device, session, plex_sessions,
'media_player', prefix,
self.name.lower().replace('-', '_'))

# pylint: disable=too-many-branches, too-many-statements
def refresh(self, device, session):
"""Refresh key device data."""
# new data refresh
Expand Down
6 changes: 3 additions & 3 deletions homeassistant/components/sensor/openevse.py
Expand Up @@ -7,7 +7,6 @@
import logging

from requests import RequestException

import voluptuous as vol

import homeassistant.helpers.config_validation as cv
Expand All @@ -16,8 +15,10 @@
from homeassistant.const import CONF_MONITORED_VARIABLES
from homeassistant.helpers.entity import Entity

_LOGGER = logging.getLogger(__name__)
REQUIREMENTS = ['openevsewifi==0.4']

_LOGGER = logging.getLogger(__name__)

SENSOR_TYPES = {
'status': ['Charging Status', None],
'charge_time': ['Charge Time Elapsed', 'minutes'],
Expand Down Expand Up @@ -54,7 +55,6 @@ def setup_platform(hass, config, add_devices, discovery_info=None):
class OpenEVSESensor(Entity):
"""Implementation of an OpenEVSE sensor."""

# pylint: disable=too-many-arguments
def __init__(self, sensor_type, charger):
"""Initialize the sensor."""
self._name = SENSOR_TYPES[sensor_type][0]
Expand Down
3 changes: 1 addition & 2 deletions homeassistant/components/vera.py
Expand Up @@ -62,7 +62,7 @@ def setup(hass, base_config):

def stop_subscription(event):
"""Shutdown Vera subscriptions and subscription thread on exit."""
_LOGGER.info("Shutting down subscriptions.")
_LOGGER.info("Shutting down subscriptions")
VERA_CONTROLLER.stop()

config = base_config.get(DOMAIN)
Expand Down Expand Up @@ -100,7 +100,6 @@ def stop_subscription(event):
return True


# pylint: disable=too-many-return-statements
def map_vera_device(vera_device, remap):
"""Map vera classes to Home Assistant types."""
import pyvera as veraApi
Expand Down