-
Notifications
You must be signed in to change notification settings - Fork 589
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 missing boards for CP 8.2.9 plus more Adafruit links #1301
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On the WisdPi boards, the description says "Introducing our...", which might be confused as something from Adafruit? Or do we have plenty of text like that? Could be made third-person instead of first-person?
I wonder if these descriptions might link to the originals. Then we could have a link at the top "From ".
I generally try and change text like that, so I just updated it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wisdpi_tiny_rp2040.md needs a little copyediting to remove first-person references. It's also quite redundant: for instance the additional WS2812 RGB LED is mentioned three times (!).
I started to propose suggestions, but it got out of hand :)
Yeah, I had copied and pasted from their website. I just revised most of it and updated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for cleanining up the copy.
The multiple links for boards is working out great! Thanks for adding that. |
You're welcome. :) |
No description provided.