-
Notifications
You must be signed in to change notification settings - Fork 5
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
Restart() method missing? #1
Comments
Hi @dkotik. Restart() was removed at the v0.3 release. There was an interesting race condition when using it and as I didn't have time to look at it, I removed it. If you want it back, and have time to debug that (use v0.2) then I'd be more than happy to review a PR 👍 |
Ok! Thank you for clarification - I just was confused by this paragraph in Readme: https://github.com/leaanthony/spinner#resuse-spinner It needs, perhaps be removed, for now? I am ok without a Restart method. |
Good catch! I will update it 👍👍👍
…On Wed, 2 Oct 2019, 17:22 Dmitry Kotik, ***@***.***> wrote:
Ok! Thank you for clarification - I just was confused by this paragraph in
Readme: https://github.com/leaanthony/spinner#resuse-spinner It needs,
perhaps be removed, for now? I am ok without a Restart method.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#1?email_source=notifications&email_token=AAO2SYELCBOELWAQEJ3NP2TQMRD5DA5CNFSM4I4KOLGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEADZ3RA#issuecomment-537370052>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAO2SYDRIVIRZERTUQL26YTQMRD5DANCNFSM4I4KOLGA>
.
|
At some point, I'll revisit this. Thanks for raising it! |
Using v0.5.3 - method "Restart" is missing?
The text was updated successfully, but these errors were encountered: