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

Execute tests on Node.js 11 #584

Closed
wants to merge 1 commit into from
Closed

Execute tests on Node.js 11 #584

wants to merge 1 commit into from

Conversation

kibertoad
Copy link

@kibertoad kibertoad commented Oct 23, 2018

Add Node.js 11 to Travis configuration

@thelinuxfoundation
Copy link

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.

After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

Regards,
CLA GitHub bot

@kibertoad
Copy link
Author

Not going to sign CLA just for this, but anyone with CLA signed is welcome to fork my branch and get this merged.

@kibertoad kibertoad closed this Oct 23, 2018
@nicolasnoble
Copy link
Member

We are actually not using Travis, so this PR would've been irrelevant anyway.

@kibertoad
Copy link
Author

@nicolasnoble Why have travis.yaml then?..

@nicolasnoble
Copy link
Member

Leftovers from another life. For the node11 support, you can follow along in #593.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants