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

Remove node 0.12 from Travis-CI. #220

Merged
merged 2 commits into from
Apr 5, 2017
Merged

Conversation

adrianba
Copy link
Contributor

@adrianba adrianba commented Apr 4, 2017

Dependencies for alexa-app no longer work on Node 0.12 so remove node 0.12 from test matrix. Run extra steps with Node 1 instead. Also updated engine version in package.json.

@tejashah88
Copy link
Member

@dblock @adrianba Since we are removing testing support for node v0.12, would it make sense to just drop node v0.12 support for this library entirely?

@dblock
Copy link
Collaborator

dblock commented Apr 5, 2017

Yes, please, do remove node 0.12 support if we can't run tests anymore. Also we do want a green build whatever that takes ;)

@adrianba
Copy link
Contributor Author

adrianba commented Apr 5, 2017

Working on it - seeing some odd errors so will set-up Travis against my fork and figure out what is going on.

Dependencies for alexa-app no longer work on Node 0.12 so remove node 0.12 from test matrix. Run extra steps with Node 1 instead.

Missing space.

Update engine.
@adrianba
Copy link
Contributor Author

adrianba commented Apr 5, 2017

Okay, seems to be working now - also updated min version in package.json.

CHANGELOG.md Outdated
@@ -4,6 +4,7 @@

* [#219](https://github.com/alexa-js/alexa-app/pull/219): Preserve session when clearing non-existent attribute - [@adrianba](https://github.com/adrianba).
* [#218](https://github.com/alexa-js/alexa-app/pull/218): Fix cert check test cases - [@adrianba](https://github.com/adrianba).
* [#220](https://github.com/alexa-js/alexa-app/pull/220): Remove Node 0.12 from Travis-CI - [@adrianba](https://github.com/adrianba).
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should say "No longer compatible with Node 0.12".

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@dblock dblock merged commit dc97022 into alexa-js:master Apr 5, 2017
@dblock
Copy link
Collaborator

dblock commented Apr 5, 2017

Thanks @adrianba.

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