Skip to content

Add support for "effects" for light (#142) - #178

Open
martaisty wants to merge 3 commits into
dawidchyrzynski:developfrom
martaisty:feature/light-effects-list
Open

Add support for "effects" for light (#142)#178
martaisty wants to merge 3 commits into
dawidchyrzynski:developfrom
martaisty:feature/light-effects-list

Conversation

@martaisty

Copy link
Copy Markdown
Contributor

Please review and feel free to suggest any good changes 🙂
The PR closes issue #142

Changes:

  • Added support for effects for HALight
  • Updated light example with effects usage
  • Covered new code with Unit tests
  • Updated existing documentation to include effects feature

@martaisty
martaisty marked this pull request as ready for review July 23, 2023 16:11
Comment thread src/device-types/HALight.cpp
@NekoCWD

NekoCWD commented Jan 3, 2024

Copy link
Copy Markdown

Thank you for your work

@martaisty
martaisty force-pushed the feature/light-effects-list branch from 4a0acbf to da98b70 Compare February 10, 2024 15:34
@martaisty
martaisty force-pushed the feature/light-effects-list branch from da98b70 to 2b08ebe Compare February 10, 2024 15:38
@martaisty

martaisty commented Feb 10, 2024

Copy link
Copy Markdown
Contributor Author

Hi @dawidchyrzynski 👋 The PR is ready for review.
BTW I had an issue with generating docs from sources, therefore feel free to regenerate it if needed.

P.S.: I managed to generate docs. However, it seems to me that Pipfile might be broken (#231)

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