-
Notifications
You must be signed in to change notification settings - Fork 3k
Fix inclusion of test related directories by build tool #11166
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
@hugueskamba, thank you for your changes. |
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.
Looks good to me.
@ARMmbed/mbed-os-tools
CI started |
Test run: FAILEDSummary: 3 of 4 test jobs failed Failed test jobs:
|
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
I restarted events, should be ready for merge soon |
@hugueskamba Please review events failures in the travis. They don't look related. I restarted it to make sure. Not certain how adding test ignore inside cbor could affect building usb files. |
@0xc0170 The failure is not related to this change. |
…a test The `unity` dir needs to be ignore differently.
26673cb
to
e507435
Compare
Force pushed to rebase from master https://github.com/ARMmbed/mbed-os/compare/26673cb8b0c8dbc55b121cd24137d14ee5fe5a80..e5074356556fdb0659b8fa2185857e370311e5c8 to potentially fix the CI build issue. |
CI restarted |
I restarted CI. Travis seems to have a problem with hook update, I restarted it |
Test run: FAILEDSummary: 2 of 11 test jobs failed Failed test jobs:
|
Waiting for pr-merge fix , cc @ARMmbed/mbed-os-test Exporters restarted |
Description
Prevent
mbed-os/components/TARGET_PSA/services/attestation/qcbor/test
to be included by the build tool during compilation.Pull request type
Reviewers
@evedon @gpsimenos