Skip to content

Bump Insteon dependencies#88514

Merged
emontnemery merged 1 commit into
home-assistant:devfrom
teharris1:insteon-deps
Feb 22, 2023
Merged

Bump Insteon dependencies#88514
emontnemery merged 1 commit into
home-assistant:devfrom
teharris1:insteon-deps

Conversation

@teharris1
Copy link
Copy Markdown
Contributor

@teharris1 teharris1 commented Feb 20, 2023 •
edited
Loading

Proposed change

Bump the Insteon dependencies to improve stability

Changelog: pyinsteon/pyinsteon@1.3.1...1.3.2
Changelog: pyinsteon/insteon-panel@0.3.1...0.3.2

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New integration (thank you!)
  • New feature (which adds functionality to an existing integration)
  • Deprecation (breaking change to happen in the future)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • Local tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I have followed the development checklist
  • The code has been formatted using Black (black --fast homeassistant tests)
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

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

  • The manifest file has all fields filled out correctly.
    Updated and included derived files by running: python3 -m script.hassfest.
  • New or updated dependencies have been added to requirements_all.txt.
    Updated by running python3 -m script.gen_requirements_all.
  • For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description.
  • Untested files have been added to .coveragerc.

To help with the load of incoming pull requests:

@home-assistant home-assistant Bot added cla-signed dependency Pull requests marked as a dependency upgrade integration: insteon small-pr PRs with less than 30 lines. by-code-owner Quality Scale: No score labels Feb 20, 2023
frenck
frenck previously requested changes Feb 21, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

I could not find the 0.3.2 tag/release of the frontend package:

CleanShot 2023-02-21 at 11 24 52

Did something went wrong?

../Frenck

@home-assistant home-assistant Bot marked this pull request as draft February 21, 2023 10:25
@home-assistant
Copy link
Copy Markdown
Contributor

home-assistant Bot commented Feb 21, 2023

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@teharris1
Copy link
Copy Markdown
Contributor Author

teharris1 commented Feb 21, 2023

Did something went wrong?

Yes, I am an idiot. I forgot to tag the release. Fixed.

@teharris1 teharris1 marked this pull request as ready for review February 21, 2023 18:31
@home-assistant home-assistant Bot requested a review from frenck February 21, 2023 18:31
@emontnemery emontnemery dismissed frenck’s stale review February 22, 2023 13:41

Release has been tagged

@emontnemery emontnemery merged commit 0f21247 into home-assistant:dev Feb 22, 2023
raman325 added a commit to raman325/home-assistant that referenced this pull request Feb 22, 2023
@raman325
Merge branch 'dev' into target
5a6744c 
* dev: (60 commits)
  Update frontend to 20230222.0 (home-assistant#88615)
  Add controller support to `zwave_js/subscribe_firmware_update_status` (home-assistant#87348)
  Bump Freebox to 1.1.0 (home-assistant#88609)
  Always include platform in `config/entity_registry/list_for_display` (home-assistant#88601)
  Add dsk option to zwave_js/add_node WS command (home-assistant#87823)
  Update zwave_js FirmwareUploadView to support controller updates (home-assistant#87239)
  Add new zwave_js WS command to parse DSK from QR code (home-assistant#87237)
  Diagnostics report for Thread networks (home-assistant#88541)
  Set default for `hass_config_yaml` fixture to "" (home-assistant#88608)
  Bump reolink-aio to 0.5.0 (home-assistant#88594)
  Bump intents package version; hassil==1.0.5; home-assistant-intents==2023.2.22 (home-assistant#88605)
  Add Reolink update entity (home-assistant#87865)
  Fix cover template: optimistic mode is ignored (home-assistant#87925)
  Fix 500 error when getting calendar events (home-assistant#88276)
  Add clarifying comment about unit of elevation (home-assistant#88489)
  Add ZHA "consumer connected" binary sensor for Xiaomi EU plugs (home-assistant#88194)
  Bump Insteon dependencies (home-assistant#88514)
  Use load_json_object in ecobee (home-assistant#88584)
  Use load_json_object in html5 (home-assistant#88586)
  Improve type hint in homeassistant trigger (home-assistant#88596)
  ...
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 23, 2023
@teharris1 teharris1 deleted the insteon-deps branch March 14, 2023 14:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@emontnemery emontnemery emontnemery approved these changes

@frenck frenck Awaiting requested review from frenck

Assignees

No one assigned

Labels

by-code-owner cla-signed dependency Pull requests marked as a dependency upgrade integration: insteon Quality Scale: No score small-pr PRs with less than 30 lines.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@teharris1 @frenck @emontnemery