-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Fix an incorrect comment in ticker API #5019
Fix an incorrect comment in ticker API #5019
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.
If that's how it behaves and nobody says it should behave differently than LGTM.
cc @pan- |
I've added 'do not merge' label to wait for outcome of #5051 |
cc @pan- LGTM |
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
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputBuild failed! |
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputTest failed! |
One target became unavailable, restarting /morph test |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
Description
Fix an incorrect warning on the use of
ticker_insert_event_us
with atimestamp
from the past. For reference look at:/hal/mbed_ticker_api.c
/TESTS/mbed_hal/ticker/main.cpp
Status
READY
@bulislaw @0xc0170