Closed bdraco closed 2 years ago
This should be the last PR for today.
If you have time, a release would be great as well as I might be able to get the remotes working after dinner.
Merging #121 (7dd3128) into master (0b11c5f) will increase coverage by
0.47%
. The diff coverage is37.50%
.
@@ Coverage Diff @@
## master #121 +/- ##
==========================================
+ Coverage 76.89% 77.36% +0.47%
==========================================
Files 12 12
Lines 779 782 +3
==========================================
+ Hits 599 605 +6
+ Misses 180 177 -3
Impacted Files | Coverage Δ | |
---|---|---|
pywizlight/scenes.py | 100.00% <ø> (ø) |
|
pywizlight/bulblibrary.py | 90.00% <33.33%> (-1.67%) |
:arrow_down: |
pywizlight/bulb.py | 83.02% <50.00%> (+1.10%) |
:arrow_up: |
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 0b11c5f...7dd3128. Read the comment docs.
Reviewed, merged and release created. Sorry for the delay 😄 - there are some timezones between us.
Add coverage for scenes
Change ValueError to
WizLightNotKnownBulb
since they are thrown for similar cases which supports https://github.com/home-assistant/core/pull/66305