Skip to content

device_tracker.asuswrt: Clean up unused connection param#12262

Merged
balloob merged 1 commit intohome-assistant:devfrom
trisk:asuswrt_unused_connection_param
Feb 9, 2018
Merged

device_tracker.asuswrt: Clean up unused connection param#12262
balloob merged 1 commit intohome-assistant:devfrom
trisk:asuswrt_unused_connection_param

Conversation

@trisk
Copy link
Copy Markdown
Contributor

@trisk trisk commented Feb 9, 2018

Description:

This removes the ap parameter for the SshConnection and TelnetConnection class methods. This parameter is used to assign the _ap attribute as a boolean, which is never used. (The ap configuration parameter check uses the mode attribute of AsusWrtDeviceScanner).

Checklist:

  • The code change is tested and works locally.

If the code communicates with devices, web services, or third-party tools:

  • [x ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

@balloob balloob merged commit 1db4df6 into home-assistant:dev Feb 9, 2018
@trisk trisk deleted the asuswrt_unused_connection_param branch February 9, 2018 23:24
@balloob balloob mentioned this pull request Feb 22, 2018
@home-assistant home-assistant locked and limited conversation to collaborators May 29, 2018
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.

3 participants