delfick / photons

Python3.6+ asyncio framework for interacting with LIFX devices
https://photons.delfick.com
MIT License
73 stars 6 forks source link

control: add support for setting the direction of the MOVE effect #111

Closed Djelibeybi closed 2 years ago

Djelibeybi commented 2 years ago

Includes an update to the docs for both the task and Interactor command to provide a little more clarity on how to set this and other options.

Also updated the test for this effect.

Signed-off-by: Avi Miller me@dje.li

delfick commented 2 years ago

Nice :)

Though assignment operator is 3.8+ and photons in 3.7+ till June next year (when python stops making security updates for it).

But whether I be lazy and just make it 3.9+ wouldn't be hard to convince me of hahaha

On Thu, 23 June 2022, 5:42 pm Avi Miller, @.***> wrote:

@Djelibeybi https://github.com/Djelibeybi requested your review on: #111 https://github.com/delfick/photons/pull/111 control: add support for setting the direction of the MOVE effect.

— Reply to this email directly, view it on GitHub https://github.com/delfick/photons/pull/111#event-6863834817, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA2V5PJI6HIJXSFKTYSF33VQQIPJANCNFSM5ZTKUS6A . You are receiving this because your review was requested.Message ID: @.***>

Djelibeybi commented 2 years ago

But whether I be lazy and just make it 3.9+ wouldn't be hard to convince me of hahaha

Home Assistant is already 3.10+ which explains why I used the operator, I guess. But this is not significant enough a change to warrant a dependency bump, so I fixed it. :)

Djelibeybi commented 2 years ago

Oh, I love the fact that GitHub now lets you commit a suggestion by clicking a button. That's very handy.

Djelibeybi commented 2 years ago

GitHub's auto-merging is actually bad. Apologies. I'll pull and make the changes properly.

delfick commented 2 years ago

GitHub's auto-merging is actually bad. Apologies. I'll pull and make the changes properly.

hhahahah

delfick commented 2 years ago

https://github.com/delfick/photons/runs/7034985825?check_suite_focus=true