-
-
Notifications
You must be signed in to change notification settings - Fork 150
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
Added support for M5Core2 in Graphics Primitives #302
Added support for M5Core2 in Graphics Primitives #302
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.
Thanks for the proposition but this sample should work for all kind of MCU. So if you want to create a specific version for M5Stack, I'm fine. In that case, create another project, reuse all the cs file (by reference in the project, not by copy/paste), and adjust as well the README and category.txt so we're clear that it's a specific sample for M5Stack
@Ellerbach this seems to be stalled... do we want to merge it anyways and rework it later? |
No merge for this one. Like we did for others, it's ok to create 2 projects so they work directly. |
bda3a96
to
8bdc94c
Compare
@jsuutari I'll close this PR open for more than a year. If you are still willing to contribute to this sample, please create a separate project just for the M5Core2. Thanks! |
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (3)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Description
Added support for M5Core2 in Graphics Primitives Sample
Motivation and Context
How Has This Been Tested?
Screenshots
Types of changes
Checklist: