-
Notifications
You must be signed in to change notification settings - Fork 3k
Print the warning messages in debug mode #8222
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
fabc880
to
e0e8bd4
Compare
Running this through CI on its own. Note: When this was attempted to be used in a rollup PR, it seems that this PR caused issues. /morph build |
Build : SUCCESSBuild number : 3339 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 2973 |
Test : FAILUREBuild number : 3147 |
Looks like networking issue in our env, will restart later |
This now confirms that the issue with the original rollup was with this PR. (Before: #8345 After: #8356) For some reason, this change reliably hits the time_start bug with networked devices, and needs to be solved. |
Please lower the priority of this PR, will look into this later next week |
Description
Warning string passed in MBED_WARN was not used, adding debug message to print it in debug mode
Pull request type