Skip to content

Conversation

@bdraco
Copy link
Member

@bdraco bdraco commented Apr 5, 2022

@bdraco
Copy link
Member Author

bdraco commented Apr 5, 2022

Looks like the CI is broken for unrelated reasons

@rytilahti
Copy link
Member

CI fix in #331

@bdraco bdraco force-pushed the restore_state_turning_back_on branch from 25620e4 to fbcf717 Compare April 5, 2022 16:23
@bdraco
Copy link
Member Author

bdraco commented Apr 5, 2022

Looks like the CI is passing sans the docs which don't look related to this change

@bdraco
Copy link
Member Author

bdraco commented Apr 5, 2022

@rytilahti If this is ok now, I was hoping to get a new release to fix home-assistant/core#69039 before beta is finished up (and before modularize merges).

@codecov-commenter
Copy link

codecov-commenter commented Apr 5, 2022

Codecov Report

Merging #330 (fbcf717) into master (a744af4) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #330      +/-   ##
==========================================
+ Coverage   82.34%   82.37%   +0.03%     
==========================================
  Files          13       13              
  Lines        1376     1379       +3     
  Branches      202      203       +1     
==========================================
+ Hits         1133     1136       +3     
  Misses        201      201              
  Partials       42       42              
Impacted Files Coverage Δ
kasa/smartbulb.py 94.44% <100.00%> (+0.10%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a744af4...fbcf717. Read the comment docs.

Copy link
Member

@rytilahti rytilahti left a comment

Choose a reason for hiding this comment

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

Thanks @bdraco, let's merge this and I'll prepare a release immediately before the modularize merge.

@rytilahti rytilahti added the bug Something isn't working label Apr 5, 2022
@rytilahti rytilahti changed the title Ensure state is restored when turning back on Ensure bulb state is restored when turning back on Apr 5, 2022
@rytilahti rytilahti merged commit 766819a into python-kasa:master Apr 5, 2022
rytilahti added a commit to rytilahti/python-kasa that referenced this pull request Apr 5, 2022
This release fixes an issue where the bulb state on led strips was not restored properly when turned back on.

[Full Changelog](python-kasa/python-kasa@0.4.2...0.4.3)

**Fixed bugs:**

- Divide by zero when HS300 powerstrip is discovered [\python-kasa#292](python-kasa#292)
- Ensure bulb state is restored when turning back on [\python-kasa#330](python-kasa#330) (@bdraco)

**Closed issues:**

- KL420L5 controls [\python-kasa#327](python-kasa#327)

**Merged pull requests:**

- Update pre-commit hooks to fix black in CI [\python-kasa#331](python-kasa#331) (@rytilahti)
- Fix test\_deprecated\_type stalling [\python-kasa#325](python-kasa#325) (@bdraco)
@rytilahti rytilahti mentioned this pull request Apr 5, 2022
rytilahti added a commit that referenced this pull request Apr 5, 2022
This release fixes an issue where the bulb state on led strips was not restored properly when turned back on.

[Full Changelog](0.4.2...0.4.3)

**Fixed bugs:**

- Divide by zero when HS300 powerstrip is discovered [\#292](#292)
- Ensure bulb state is restored when turning back on [\#330](#330) (@bdraco)

**Closed issues:**

- KL420L5 controls [\#327](#327)

**Merged pull requests:**

- Update pre-commit hooks to fix black in CI [\#331](#331) (@rytilahti)
- Fix test\_deprecated\_type stalling [\#325](#325) (@bdraco)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TPLink Kasa Integration: KL430 - Persistent Effect Behavior

3 participants