Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add rainbowio #5534

Merged
merged 2 commits into from
Nov 3, 2021
Merged

Add rainbowio #5534

merged 2 commits into from
Nov 3, 2021

Conversation

jepler
Copy link
Member

@jepler jepler commented Nov 1, 2021

We have rainbowio manually disabled on 5 boards. Reenable on these two, where it fits.

@jepler jepler marked this pull request as ready for review November 1, 2021 17:53
@jepler jepler requested a review from dhalbert November 1, 2021 17:53
Copy link
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for trying some of these. The simmel is a special-purpose nRF contact-tracing board, so everything irrelevant was turned off. So I'd say just do the sensebox_mcu

@jepler
Copy link
Member Author

jepler commented Nov 2, 2021

@kattni what do you think? this "simmel" board doesn't have a neopixel built in nor is it intended that anyone would add one on.

@kattni
Copy link

kattni commented Nov 2, 2021

@jepler If there's definitely no reason to use RGB LEDs with the simmel board, then I agree, it's probably not worth including. That leaves that space available for future modules that make more sense for it.

@jepler jepler requested a review from dhalbert November 3, 2021 00:01
@dhalbert dhalbert merged commit 38c3816 into adafruit:main Nov 3, 2021
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