Skip to content

Conversation

@jon-hall
Copy link
Contributor

… wrapped if they are encountered elsewhere in the object being promisified.

Fixes #13.

@maxkorp
Copy link
Contributor

maxkorp commented Nov 30, 2015

Seems like the test errors on this were just network glitches on travis, starting those over again 👍

@maxkorp
Copy link
Contributor

maxkorp commented Nov 30, 2015

Ah, so, it's a module install failure against node 0.8, so pending hearing back from other maintainers re: dropping 0.8 support (its a weeee bit out of date at this point), this should be good to merge 👍

@maxkorp
Copy link
Contributor

maxkorp commented Nov 30, 2015

Can you rebase this on top of master please, to get the updated version matrix (it doesnt appear there were any other changes, so a rebase should be safe).

… wrapped if they are encountered elsewhere in the object being promisified.
@jon-hall jon-hall force-pushed the feature/unwrapped-function-cache-bug branch from f7279a6 to fe2b27c Compare November 30, 2015 16:32
@jon-hall
Copy link
Contributor Author

Cheers for updating the test config so quickly 😃, rebased and pushed and it looks like everything passes now.

@maxkorp
Copy link
Contributor

maxkorp commented Nov 30, 2015

Shoop da woop!

maxkorp added a commit that referenced this pull request Nov 30, 2015
…-bug

Fix issue with async functions being cached as-is, and so not getting…
@maxkorp maxkorp merged commit 0435d61 into nodegit:master Nov 30, 2015
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.

2 participants