Skip to content

Upgrade sqlalchemy to 1.2.17#21020

Merged
fabaff merged 4 commits into
devfrom
upgrade-sqlalchemy
Feb 13, 2019
Merged

Upgrade sqlalchemy to 1.2.17#21020
fabaff merged 4 commits into
devfrom
upgrade-sqlalchemy

Conversation

@fabaff
Copy link
Copy Markdown
Member

@fabaff fabaff commented Feb 12, 2019

Description:

Changelog: https://docs.sqlalchemy.org/en/latest/changelog/changelog_12.html#change-1.2.17

Example entry for configuration.yaml (if applicable):

recoder:

sensor:
  - platform: sql
    queries:
      - name: Sun state
        query: "SELECT * FROM states WHERE entity_id = 'sun.sun' ORDER BY state_id DESC LIMIT 1;"
        column: 'state'

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.

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

  • New dependencies have been added to the REQUIREMENTS variable (example).
  • New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.

@ghost ghost assigned dgomes Feb 13, 2019
@dgomes
Copy link
Copy Markdown
Contributor

dgomes commented Feb 13, 2019

Good to go!

@fabaff fabaff merged commit 8db8a58 into dev Feb 13, 2019
@ghost ghost removed the in progress label Feb 13, 2019
@delete-merged-branch delete-merged-branch Bot deleted the upgrade-sqlalchemy branch February 13, 2019 11:30
@balloob balloob mentioned this pull request Feb 20, 2019
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.

3 participants