Skip to content

Conversation

kumar303
Copy link
Contributor

Fixes #322

@coveralls
Copy link

coveralls commented Jun 27, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling d2280ef on kumar303:timeout into 924a93f on mozilla:master.

@rpl
Copy link
Member

rpl commented Jun 27, 2016

@kumar303 r+

question: do we currently print the current timeout time to the user? I'm thinking that with a longer default timeout, it could be reasonable to give to the user more informations about how much time is could be waiting for even when web-ext is not running in verbose mode.

@kumar303
Copy link
Contributor Author

I see what you mean. I added some logging so at least the user knows web-ext is trying to establish a connection.

@coveralls
Copy link

coveralls commented Jun 27, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 436ccd4 on kumar303:timeout into f256739 on mozilla:master.

@rpl
Copy link
Member

rpl commented Jun 27, 2016

@kumar303 👍

r+

@kumar303 kumar303 merged commit 830065b into mozilla:master Jun 27, 2016
@kumar303 kumar303 deleted the timeout branch June 27, 2016 19:02
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.

3 participants