-
Notifications
You must be signed in to change notification settings - Fork 3k
Added examples uvisor-thread and uvisor-number-store #3306
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
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.
LGTM
@mazimkhan Could you please edit the PR comment and fill out some of the sections of the template? Or just remove the non relevant sections, thanks!
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputExample Build failed! |
The example build failure didn't get captured by the public results, so here's the failure from CI:
@mazimkhan Are you sure the |
@AlessandroA can you please check uvisor build errors. |
retest uvisor |
target EFM32GG_STK3700 removed after dicussion with @AlessandroA |
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputExample Build failed! |
/morph test |
1 similar comment
/morph test |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
CI looks good now from the uvisor perspective. |
Notes:
Description
Added examples uvisor-thread and uvisor-number-store
https://github.com/ARMmbed/mbed-os-example-uvisor-thread
https://github.com/ARMmbed/mbed-os-example-uvisor-number-store
Status
READY
Migrations
If this PR changes any APIs or behaviors, give a short description of what API users should do when this PR is merged.
NO
@bridadan please review