Is there anything else I should know about the code?
This project provides access to USB connected presence lights from multiple vendors. It attempts to put a common interface in front of different lights with their varying capabilities.
The test cases run regardless of the presence or absence of a light.
Are there any areas you would like me to focus on?
The ABC class busylight.lights.usblight.USBLight and concrete implementations
The threading model is sort of a hack. Some lights need active management so a thread is necessary.
The Manager class busylight.manager.Manager is pretty raw
What is the name of your project?
BusyLight for Humans™
What is the repo URL? https://github.com/JnyJny/busylight
Is there anything else I should know about the code? This project provides access to USB connected presence lights from multiple vendors. It attempts to put a common interface in front of different lights with their varying capabilities.
The test cases run regardless of the presence or absence of a light.
Are there any areas you would like me to focus on?
busylight.lights.usblight.USBLight
and concrete implementationsbusylight.manager.Manager
is pretty rawHave I reviewed this project previously? No
Thank you for your time!