Skip to content

Update adafruit_feather_rp2040.md#850

Merged
makermelissa merged 2 commits into
adafruit:mainfrom
sctv:patch-2
Jan 26, 2022
Merged

Update adafruit_feather_rp2040.md#850
makermelissa merged 2 commits into
adafruit:mainfrom
sctv:patch-2

Conversation

@sctv
Copy link
Copy Markdown
Contributor

@sctv sctv commented Jan 24, 2022

Updated that rp2040 based feather can use an Arduino core. Welcome feedback on how to phrase it better.
A similar line should be included in all rp2040 based boards.
Thanks.

Updated that rp2040 based feather can use an Arduino core.  Welcome feedback on how to phrase it.
A similar line should be included in all rp2040 based boards. 
Thanks.
@jepler
Copy link
Copy Markdown
Contributor

jepler commented Jan 24, 2022

Thanks, this would be a good update to make.

Because we like the text for Adafruit products to match what's in the shop, I've raised this issue internally. I'll get back to you about this, and if I don't you should feel free to ping me about it in a few days.

Copy link
Copy Markdown
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

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

It looks like you have some duplicate text in there. Want to fix it?

Have deleted duplicate text. 
I have now realized the text has changed in the product page.  On [https://www.adafruit.com/product/4884] the arduino support is mentioned, so it seems more a sync issue from the product page.
@sctv sctv closed this Jan 25, 2022
@sctv sctv reopened this Jan 25, 2022
@sctv
Copy link
Copy Markdown
Contributor Author

sctv commented Jan 25, 2022

@jepler @makermelissa will sync this description up with the product page rp2040 feather tomorrow. Still getting my head around git/hub/branches/pr's etc.
thanks for your patience. Have allowed edits by maintainers on patch-2.

Copy link
Copy Markdown
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks for making the changes. Also we prefer to assist others in learning how to use GitHub instead of just doing it for them.

@makermelissa makermelissa merged commit 62f6400 into adafruit:main Jan 26, 2022
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