-
Notifications
You must be signed in to change notification settings - Fork 4
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
ES6 #5
Conversation
.then((_) => { | ||
let promises = [] | ||
|
||
const shuffle = knuth_shuffle.knuthShuffle |
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.
cupid shuffle :)
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.
uhh... okay?
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.
* @return {Object} Cleaned (if applicable) value. | ||
*/ | ||
clean_value (value) { | ||
debug('clean_value') |
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.
would you want to put the value in the debug statement?
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.
not really... the reason I have all those debug
statements was to track the flow of things for a while. they can be expanded in the future, but not interested in doing that now
the test delays for 5, but sometimes CI runs fast enough (and locally) that the delay is exactly 5 as well. let's just bring it down a har
+1 look cool |
Rewrote the module in ES6. When published, it builds the source to a common js module so it is still usable by
0.10
.