Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Set min node.js version to 6.9 #918

Closed
wants to merge 1 commit into from

Conversation

webmaster128
Copy link
Contributor

In #861 support for nodejs < 6 was dropped. However, the minimal version was to 6.11.5 for no clear reason. This excludes version v6.11.4 from current Ubuntu 17.10 as well as version 6.10.3 running on AWS Lambda.

If features newer then 6.0.0 are needed, we should only set the minor version, not the patch level, e.g. 6.11.0 instead of 6.11.5 because Linux distributions often apply patches to a fixed version, so v6.11.4 on Ubuntu is likely to have all security fixes that are in 6.11.5.

@mastilver
Copy link
Collaborator

I see you opened an issue there: webpack/webpack#6579

Can you please set min version to 6.9 as it seems to be the result of the conversation there

In jantimon#861 support for
nodejs < 6 was dropped. However, the minimal version was to 6.11.5 for
no clear reason. This excludes version v6.11.4 from current Ubuntu 17.10
as well as version 6.10.3 running on AWS Lambda.

See also discussion in webpack/webpack#6579
@webmaster128
Copy link
Contributor Author

Thanks for feedback, updated

@webmaster128 webmaster128 changed the title Set min node.js version to 6.0.0 Set min node.js version to 6.9 Apr 1, 2018
jantimon pushed a commit that referenced this pull request Apr 3, 2018
In #861 support for
nodejs < 6 was dropped. However, the minimal version was to 6.11.5 for
no clear reason. This excludes version v6.11.4 from current Ubuntu 17.10
as well as version 6.10.3 running on AWS Lambda.
See also discussion in webpack/webpack#6579

#918
@jantimon
Copy link
Owner

jantimon commented Apr 3, 2018

I also set the travis version to 6.9 and will merge it as soon as travis is happy :)

jantimon pushed a commit that referenced this pull request Apr 3, 2018
In #861 support for
nodejs < 6 was dropped. However, the minimal version was to 6.11.5 for
no clear reason. This excludes version v6.11.4 from current Ubuntu 17.10
as well as version 6.10.3 running on AWS Lambda.
See also discussion in webpack/webpack#6579

#918
@jantimon
Copy link
Owner

jantimon commented Apr 3, 2018

Merged into 3.2.0

@jantimon jantimon closed this Apr 3, 2018
@webmaster128
Copy link
Contributor Author

Thanks!

@lock
Copy link

lock bot commented May 31, 2018

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators May 31, 2018
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants