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

Wu lee/use current compile #27

Closed
wants to merge 3 commits into from

Conversation

wu-lee
Copy link

@wu-lee wu-lee commented Apr 4, 2016

Uses couchdb-compile@~1.6.2 as per @argl's original pull request, but also includes a fix to make the tests pass.

I can confirm this version does not have the problem described in #26. Due to deficiencies in NodeJS/nodeunit's deepEqual() function, I have not written a test which catches that issue, since I am currently unsure of what the best work around would be.

@wu-lee
Copy link
Author

wu-lee commented Apr 4, 2016

Node 0.8 doesn't build for reasons I confess I don't understand. Specifically, [email protected] and glob@^6.0.4 are valid versions.

@argl
Copy link
Contributor

argl commented Apr 4, 2016

strange indeed

@wu-lee
Copy link
Author

wu-lee commented Apr 7, 2016

I think it is a problem with node v0.8 shipping with an npm which doesn't understand caret versions. PR #29 may fix it.

@jo
Copy link
Owner

jo commented Apr 7, 2016

Hi @wu-lee, thank you so much for your work! I have cherry-picked your commit, its in master now: 041d0aa

@jo jo closed this Apr 7, 2016
@wu-lee wu-lee deleted the wu-lee/use_current_compile branch April 8, 2016 14:48
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.

3 participants