X Tutup
Skip to content

Update common lib#184

Merged
rosen-vladimirov merged 1 commit intomasterfrom
vladimirov/print-msg-after-timeout
Jan 16, 2015
Merged

Update common lib#184
rosen-vladimirov merged 1 commit intomasterfrom
vladimirov/print-msg-after-timeout

Conversation

@rosen-vladimirov
Copy link
Copy Markdown
Contributor

In common lib the following changes are applied:
Add function to print message after timeout: add new function to helpers.ts which prints a message on the same line (using printInfoMessageOnSameLine method from the same helpers.ts) after a specific timeout had passed. Using future to guarantee that the function will not finish until timeout is passed and message is printed.

Fix missing ";" in options.ts.

@ns-bot
Copy link
Copy Markdown

ns-bot commented Jan 16, 2015

Test PASSed.

In common lib the following change is applied:
Add function to print message after timeout - add new function to helpers.ts which prints a message on the same line (using printInfoMessageOnSameLine method from the same helpers.ts) after a specific timeout had passed. Using future to guarantee that the function will not finish until timeout is passed and message is printed.

Fix missing ";" in options.ts.
@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/print-msg-after-timeout branch from 7958b09 to 7c38899 Compare January 16, 2015 09:02
@rosen-vladimirov rosen-vladimirov added this to the v0.5 milestone Jan 16, 2015
@rosen-vladimirov rosen-vladimirov self-assigned this Jan 16, 2015
@ns-bot
Copy link
Copy Markdown

ns-bot commented Jan 16, 2015

Test PASSed.

@teobugslayer
Copy link
Copy Markdown
Contributor

👍

1 similar comment
@ErjanGavalji
Copy link
Copy Markdown
Contributor

👍

rosen-vladimirov added a commit that referenced this pull request Jan 16, 2015
@rosen-vladimirov rosen-vladimirov merged commit b50ed71 into master Jan 16, 2015
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/print-msg-after-timeout branch January 16, 2015 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup