Skip to content
This repository was archived by the owner on Feb 2, 2021. It is now read-only.

Conversation

@TsvetanMilanov
Copy link
Contributor

After removing the fibers we forgot to await the result from the prompt before calling the unmuteStdout method. After this method is invoked the input from the terminal is not accepted.

@justcodebuilduser
Copy link

💔

Copy link
Contributor

@Mitko-Kerezov Mitko-Kerezov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks okay

After removing the fibers we forgot to await the result from the prompt before calling the unmuteStdout method. After this method is invoked the input from the terminal is not accepted.
@justcodebuilduser
Copy link

❤️

TsvetanMilanov added a commit to NativeScript/nativescript-cli that referenced this pull request Jan 18, 2017
After removing the fibers we forgot to await the result from the prompt before calling the unmuteStdout method. After this method is invoked the input from the terminal is not accepted.

Common lib reference: telerik/mobile-cli-lib#877
@TsvetanMilanov TsvetanMilanov merged commit 818b9ae into master Jan 18, 2017
@TsvetanMilanov TsvetanMilanov deleted the milanov/fix-prompter branch January 18, 2017 13:47
TsvetanMilanov added a commit to NativeScript/nativescript-cli that referenced this pull request Jan 18, 2017
After removing the fibers we forgot to await the result from the prompt before calling the unmuteStdout method. After this method is invoked the input from the terminal is not accepted.

Common lib reference: telerik/mobile-cli-lib#877
TsvetanMilanov added a commit to NativeScript/nativescript-cli that referenced this pull request Jan 18, 2017
After removing the fibers we forgot to await the result from the prompt before calling the unmuteStdout method. After this method is invoked the input from the terminal is not accepted.

Common lib reference: telerik/mobile-cli-lib#877
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants