-
-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Fixed several typos in promise.js #3172
Conversation
Fixes #3171 |
@@ -2165,7 +2165,7 @@ const SIMPLE_SCHEDULER = new SimpleScheduler; | |||
/** | |||
* Handles the execution of scheduled tasks, each of which may be an | |||
* asynchronous operation. The control flow will ensure tasks are executed in | |||
* the ordered scheduled, starting each task only once those before it have | |||
* the ordered schedule, starting each task only once those before it have |
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.
in the order scheduled
Fixed, @jleyba please review |
Currently build in Travis failed because of "No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself." - see https://travis-ci.org/SeleniumHQ/selenium/jobs/179590026#L1780. So, it's definitely not because of the typos fixing in this PR. @jleyba please review again and merge if it's ok! |
@mach6 @lukeis @titusfortner please review |
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.
LGTM
Thanks! |
X
in the preceding checkbox, I verify that I have signed the Contributor License Agreement