Skip to content
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

Merged
merged 6 commits into from
Dec 3, 2016

Conversation

Marketionist
Copy link
Contributor

@Marketionist
Copy link
Contributor Author

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
Copy link
Contributor

Choose a reason for hiding this comment

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

in the order scheduled

@Marketionist
Copy link
Contributor Author

Fixed, @jleyba please review

@Marketionist
Copy link
Contributor Author

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!

@Marketionist
Copy link
Contributor Author

@mach6 @lukeis @titusfortner please review

@lukeis lukeis added the C-nodejs label Dec 2, 2016
Copy link
Member

@mach6 mach6 left a comment

Choose a reason for hiding this comment

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

LGTM

@mach6 mach6 merged commit 2984b13 into SeleniumHQ:master Dec 3, 2016
@mach6
Copy link
Member

mach6 commented Dec 3, 2016

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants