Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for wall switches #139

Merged
merged 1 commit into from
Feb 23, 2022
Merged

Conversation

bdraco
Copy link
Contributor

@bdraco bdraco commented Feb 22, 2022

  • Wall Switches were showing effects but they don't have them

  • Also adds tests for hero device and discovery since they would conflict otherwise

@bdraco bdraco force-pushed the test_more_devices branch 3 times, most recently from d87b256 to 97e243e Compare February 22, 2022 23:57
@codecov-commenter
Copy link

codecov-commenter commented Feb 22, 2022

Codecov Report

Merging #139 (e5fc5f7) into master (e86d1ea) will increase coverage by 1.54%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #139      +/-   ##
==========================================
+ Coverage   89.41%   90.96%   +1.54%     
==========================================
  Files          12       12              
  Lines         869      874       +5     
==========================================
+ Hits          777      795      +18     
+ Misses         92       79      -13     
Impacted Files Coverage Δ
pywizlight/bulblibrary.py 100.00% <100.00%> (ø)
pywizlight/models.py 100.00% <0.00%> (+7.69%) ⬆️
pywizlight/discovery.py 98.30% <0.00%> (+20.33%) ⬆️

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 e86d1ea...e5fc5f7. Read the comment docs.

@bdraco bdraco marked this pull request as ready for review February 22, 2022 23:58
@bdraco bdraco changed the title Add tests for hero device and discovery Add support for wall switches Feb 23, 2022
bdraco added a commit to bdraco/home-assistant that referenced this pull request Feb 23, 2022
- The dimmer switch do not support effects

- Requires sbidy/pywizlight#139
@bdraco
Copy link
Contributor Author

bdraco commented Feb 23, 2022

@sbidy This one is going to need another release when you get a chance.

Thanks!

@sbidy sbidy merged commit 3b42543 into sbidy:master Feb 23, 2022
@sbidy
Copy link
Owner

sbidy commented Feb 23, 2022

Done - thank you!

@bdraco
Copy link
Contributor Author

bdraco commented Feb 23, 2022

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants