Skip to content

Push pyads to 3.0.7#21216

Merged
MartinHjelmare merged 7 commits into
home-assistant:devfrom
carstenschroeder:push-pyads-to-3.0.7
Feb 19, 2019
Merged

Push pyads to 3.0.7#21216
MartinHjelmare merged 7 commits into
home-assistant:devfrom
carstenschroeder:push-pyads-to-3.0.7

Conversation

@carstenschroeder
Copy link
Copy Markdown
Contributor

@carstenschroeder carstenschroeder commented Feb 19, 2019

Description:

ADS component did not work under Hass.io. pyads 3.0.7 corrects build issues under Alpine Linux. Minor adjustment were needed to adapt component to new release.

Related issue (if applicable): fixes https://github.com/home-assistant/hassio-homeassistant/issues/25

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.

@ghost ghost added the in progress label Feb 19, 2019
except pyads.ADSError:
_LOGGER.error(
"Could not connect to ADS host (netid=%s, port=%s)", net_id, port)
"Could not connect to ADS host (netid=%s, ip=%s, port=%s)", net_id, ip_address, port)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

line too long (97 > 79 characters)

@carstenschroeder
Copy link
Copy Markdown
Contributor Author

@MartinHjelmare Could you have a look at this PR? It's a bugfix to enable ADS on Hass.io.

@MartinHjelmare MartinHjelmare added this to the 0.88.0 milestone Feb 19, 2019
@MartinHjelmare MartinHjelmare merged commit df8589c into home-assistant:dev Feb 19, 2019
@ghost ghost removed the in progress label Feb 19, 2019
@carstenschroeder carstenschroeder deleted the push-pyads-to-3.0.7 branch February 19, 2019 18:04
balloob pushed a commit that referenced this pull request Feb 19, 2019
* Push to pyads 3.0.7

* Correct too long line
@balloob balloob mentioned this pull request Feb 20, 2019
thibmaek added a commit to thibmaek/home-assistant that referenced this pull request Feb 21, 2019
* upstream/dev: (948 commits)
  Prevent partial custom component overlays (home-assistant#21070)
  Alarm trigger support for Point (home-assistant#21207)
  Upgrade opensensemap-api to 0.1.4 (home-assistant#21240)
  Remove constraint from regex (home-assistant#21239)
  Updated frontend to 20190220.0
  Support OpenWRT 18.06 in luci device tracker (home-assistant#21236)
  Bump zigpy (home-assistant#21203)
  Fix bug in ZHA and tweak non sensor channel logic (home-assistant#21234)
  Don't dispatch to components when there are no channels for ZHA sensors (home-assistant#21223)
  Add zone and reps for Xiaomi vacuum (home-assistant#19777)
  Fix an Ambient PWS exception when location info is missing (home-assistant#21220)
  Add self to integration sensor and utility_meter (home-assistant#21226)
  Prevent invalid context from crashing (home-assistant#21231)
  Update pyhomematic to 0.1.56 (home-assistant#21227)
  Fix the build (home-assistant#21229)
  Updated frontend to 20190219.0
  Add yeelight flow action support (home-assistant#21195)
  Refactor ZHA listeners into channels (home-assistant#21196)
  Fix Homematic IP Cloud configuration (home-assistant#21202)
  Push pyads to 3.0.7 (home-assistant#21216)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pip package "pyads" not correctly installed (qemux86-64)

5 participants