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 Unsupported Versions of Node.js from .travis.yml #338

Closed
nelsonic opened this issue Feb 8, 2020 · 3 comments
Closed

Remove Unsupported Versions of Node.js from .travis.yml #338

nelsonic opened this issue Feb 8, 2020 · 3 comments

Comments

@nelsonic
Copy link
Member

nelsonic commented Feb 8, 2020

At present we have four versions of Node.js in our .travis.yml file and each one is tested by Travis CI on each commit:

language: node_js
node_js:
- "8"
- "10"
- "11"
- "node"

Given that both Node version 8 and 11 are no longer supported/maintained, I feel we should remove them.
see: https://nodejs.org/en/about/releases
image

Does anyone have any objection to the removal of these older versions of Node from .travis.yml?

@nelsonic
Copy link
Member Author

nelsonic commented Feb 8, 2020

According to this issue hapijs/hapi#4012, the BDFL of Hapi has decided to "Drop node 10"
because "I want to use node v12 features" ...
 I want to use node v12 features

So even though Node.js v10 is still maintained by the OpenJS Foundation / Node.js Community,
Version 19 of Hapi.js has introduced breaking changes so everyone is forced to upgrade.
This is not a "bad" thing because v12 does introduce numerous performance improvements including a new version of V8. So there does not appear to be any downside to upgrading. It's just a hassle some people weren't expecting, but hopefully not one that will take more than a few minutes of DevOps. 💭

See:

What does that mean for us?

It means we need to release a new major version if we update our devDependency to Hapi v19.

@nelsonic
Copy link
Member Author

nelsonic commented Feb 8, 2020

Given that Hapi 19 no longer supports Nod.js version 10.
I'm thinking of adding a line to the Compatibility section of the README.md:

image

e.g:
hapi-auth-jwt2 version 9.x.x is compatible with Hapi.js 19.x.x which only supports Node.js 12+.
While hapi-auth-jwt2 version 9.0.0 does not have any code changes from v8.8.1 (so there should not be any need to update your code that uses this plugin), we felt it was prudent to make it clear to people that Hapi.js (the core framework) has dropped support for Node.js 10 and they should treat this package as a no longer supporting older versions of Node.

@nelsonic
Copy link
Member Author

nelsonic commented Feb 8, 2020

Unsupported versions of Node.js removed and note about compatibility added #337

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

No branches or pull requests

1 participant