enwi / hueplusplus

A simple C++ library to control Philips Hue lights on Linux, MacOS, Windows, Espressif ESP32 SDK and Arduino. Full documentation at
https://enwi.github.io/hueplusplus/
GNU Lesser General Public License v3.0
55 stars 22 forks source link

Add Entertainment Mode support #65

Closed enwi closed 3 years ago

enwi commented 3 years ago

closes #60

codecov[bot] commented 3 years ago

Codecov Report

Merging #65 (ce5cfda) into development (7d74e94) will decrease coverage by 1.34%. The diff coverage is 35.68%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development      #65      +/-   ##
===============================================
- Coverage        83.22%   81.87%   -1.35%     
===============================================
  Files              100      101       +1     
  Lines             8707     8864     +157     
===============================================
+ Hits              7246     7257      +11     
- Misses            1461     1607     +146     
Impacted Files Coverage Δ
include/hueplusplus/Bridge.h 100.00% <ø> (ø)
include/hueplusplus/ColorHueStrategy.h 100.00% <ø> (ø)
include/hueplusplus/Light.h 96.62% <ø> (ø)
include/hueplusplus/LinHttpHandler.h 0.00% <ø> (ø)
include/hueplusplus/SimpleColorHueStrategy.h 100.00% <ø> (ø)
src/EntertainmentMode.cpp 0.00% <0.00%> (ø)
test/test_SimpleColorTemperatureStrategy.cpp 100.00% <ø> (ø)
test/mocks/mock_Light.h 26.66% <33.33%> (ø)
src/Bridge.cpp 71.91% <47.05%> (-11.67%) :arrow_down:
src/Light.cpp 75.60% <100.00%> (ø)
... and 10 more

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 7d74e94...ce5cfda. Read the comment docs.

lgtm-com[bot] commented 3 years ago

This pull request introduces 11 alerts when merging 9c411edad31a48e24b3200c58613f240a2a76ebb into 2b776faa13fdfc3272173d097698469d7261d251 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 11 alerts when merging d8335e63d41f238ec86db2e6160b84091fe649bb into 2b776faa13fdfc3272173d097698469d7261d251 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 11 alerts when merging 340274cc4d38d7777218538678e451a8ee033970 into 2b776faa13fdfc3272173d097698469d7261d251 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 11 alerts when merging ce5cfda6434b0a749ae45e32842f7f48d3a41913 into 7d74e949d64d64a184be2572dd055bda9e4d77a3 - view on LGTM.com

new alerts: