Skip to content

Use lastest version of jsdom to fix build issues with newer versions of Node#47

Merged
voter101 merged 1 commit intoarkency:masterfrom
WMeldon:master
Nov 6, 2015
Merged

Use lastest version of jsdom to fix build issues with newer versions of Node#47
voter101 merged 1 commit intoarkency:masterfrom
WMeldon:master

Conversation

@WMeldon
Copy link
Copy Markdown
Contributor

@WMeldon WMeldon commented Oct 6, 2015

I was having similar issues as the ones detailed here #45. The issue is directly related to the contextify package that jsdom was using. The latest version no longer uses that package (jsdom/jsdom#1188)

@kangkyu
Copy link
Copy Markdown

kangkyu commented Nov 5, 2015

Please merge if working 👍

voter101 added a commit that referenced this pull request Nov 6, 2015
Use lastest version of jsdom to fix build issues with newer versions of Node
@voter101 voter101 merged commit a70fbfc into arkency:master Nov 6, 2015
@voter101
Copy link
Copy Markdown
Contributor

voter101 commented Nov 6, 2015

Tests doesn't work. Sorry I merged it so fast without actually checking. Ref. #50

@voter101
Copy link
Copy Markdown
Contributor

voter101 commented Nov 6, 2015

Fixed in d7b7a45

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants