Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork49
Add Nicla Sense ME to compilation checks#77
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
per1234 left a comment
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.
Please investigate the compilation failures.
In cases where the sketch is simply not compatible with the board, configure the workflow so that it won't compile the sketch for the Nicla Sense ME. Seethe review on the other PR for guidance on how to accomplish that.
If a compilation failure is caused by a deficiency in the sketch that can be resolved by either a change to the "Arduino Mbed OS Nicla Boards" core, or a change to the sketch that does not have a harmful effect on the approachability of the sketch for beginners, then it is OK to leave the workflow configured to compile the sketch. However, you should be prepared to take action to resolve the failure in the near future so that this PR doesn't result in the repository having a perpetually failing CI system.
Partlycloses#75