Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Drop support for Node.js 8 #1481

Merged
merged 1 commit into from
Nov 3, 2019
Merged

Drop support for Node.js 8 #1481

merged 1 commit into from
Nov 3, 2019

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Nov 3, 2019

Since it's EOL as of 2019-12-31:
https://nodejs.org/en/about/releases/

...and if we can't support it for the full Neutrino 9 lifecycle, it's best we don't support it from the start at all.

The Travis matrix has also been updated to:

  • test against Node.js 13
  • only test against Node.js 12 on master to reduce the time taken for Travis to complete on PRs/other branches (we only get 5 concurrent builds, and previously there were 6 jobs in the matrix)

Since it's EOL as of 2019-12-31::
https://nodejs.org/en/about/releases/

...and if we can't support it for the full Neutrino 9 lifecycle, it's
best we don't support it from the start at all.

The Travis matrix has also been updated to:
* test against Node.js 13
* only test against Node.js 12 on `master` to reduce the time taken for
  Travis to complete on PRs/other branches (we only get 5 concurrent
  builds, and previously there were 6 jobs in the matrix)
@edmorley edmorley added this to the Neutrino 9 milestone Nov 3, 2019
@edmorley edmorley self-assigned this Nov 3, 2019
@edmorley edmorley requested a review from a team November 3, 2019 13:53
Copy link
Contributor

@timkelty timkelty left a comment

Choose a reason for hiding this comment

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

👍💯

@edmorley edmorley merged commit 0c0b207 into neutrinojs:master Nov 3, 2019
@edmorley edmorley deleted the edmorley-drop-node-8 branch November 3, 2019 14:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

2 participants