-
Notifications
You must be signed in to change notification settings - Fork 48
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
Async Crypto Endeavour #41
Conversation
f7424c5
to
138a3d4
Compare
Got all tests passing locally with a small fix to ipfs-repo which I'm creating a PR for now |
PR is up: ipfs/js-ipfs-repo#102 |
This is the last giant to defeat in order to finish the Async Crypto PR Mountain :) |
a0ea59b
to
540af9f
Compare
return Boolean(nextTask) | ||
}, | ||
(next) => { | ||
log('generating') | ||
log('got task', nextTask) |
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.
two logs next to each other, better coalesce to one
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.
ahhhh, there I don't wait for you one single time..
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.
:)
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.
fixed in master
No description provided.