Skip to content

Update to node 5.x.x addresses Babel 6 issues#936

Merged
quicksnap merged 2 commits intoerikras:masterfrom
andrewmclagan:master
Feb 20, 2016
Merged

Update to node 5.x.x addresses Babel 6 issues#936
quicksnap merged 2 commits intoerikras:masterfrom
andrewmclagan:master

Conversation

@andrewmclagan
Copy link
Collaborator

The pull request to update Babel 6.x.x raises the old ReferenceError: _ApiClient is not defined error on fresh installation, with node 4.2.4.

All tests are green and everything seems fine with node 5.6.0. Think its time to upgrade... Babel 6 plays nicer with 5.x.x anyway.

see PR: #935

The pull request to update Babel `6.x.x` raises the old `ReferenceError: _ApiClient is not defined` error on fresh installation, with node `4.2.4`.

All tests are green and everything seems fine with node `5.6.0`. Think its time to upgrade... Babel 6 plays nicer with 5.x.x anyway.

see PR: #935
@josmardias
Copy link
Contributor

Are you sure that it will not work with node version 5 <= x < 5.6.0 ?

@josmardias
Copy link
Contributor

Btw, I'm already using node v5 through nvm
I have a .nvmrc file with "v5" contents inside it

@andrewmclagan
Copy link
Collaborator Author

I'd say it would work with 5.x.x although only tested on 5.6.x

@quicksnap
Copy link
Collaborator

Could you add node 5 to the travis test suite? Let's see if it passes..

updated travis test suite to include 5.x.x
@andrewmclagan
Copy link
Collaborator Author

sure. done

quicksnap added a commit that referenced this pull request Feb 20, 2016
Update to node 5.x.x addresses Babel 6 issues
@quicksnap quicksnap merged commit fa0b59e into erikras:master Feb 20, 2016
@quicksnap
Copy link
Collaborator

Thanks!

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