-
Notifications
You must be signed in to change notification settings - Fork 3k
Correct percent printing #2859
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
Correct percent printing #2859
Conversation
Fixes #2831 |
Have you considered starting the percentage at the amount already built? This gives you a good feeling for the amount of build time you would have hit had you built from scratch.
|
1362e60
to
8e2acd7
Compare
@geky Done. It's even simpler. |
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.
Very cool : )
I have not written the regression test yet :) |
b0d091d
to
54eeba9
Compare
Regression tests written. |
@theotherjimmy can you resolve conflict |
54eeba9
to
860ba3a
Compare
@sg- Rebased. |
Description
Correct incremental percent printing.
Status
READY
Migrations
Nope
Todos
Steps to test or reproduce
$ touch mbed-os/hal/api/Callback.h
$ mbed compile
New output