Skip to content

Nodejs upgrade #2228

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

Merged
merged 2 commits into from
Aug 12, 2019
Merged

Nodejs upgrade #2228

merged 2 commits into from
Aug 12, 2019

Conversation

smondal
Copy link
Contributor

@smondal smondal commented Aug 12, 2019

Here is the change log from Node 6 to 8. No other changes required except version.
As Node Js still support 8, So I am changing from 6 to 8.
https://github.com/nodejs/wiki-archive/blob/master/Breaking-changes-between-v6-LTS-and-v8-LTS.md

@gauravtiwari gauravtiwari merged commit b4057d7 into rails:master Aug 12, 2019
@gauravtiwari
Copy link
Member

thanks @smondal

@frenkel
Copy link

frenkel commented Dec 13, 2019

Would it be possible to set 8.10 as a minimum? Ubuntu LTS doesn't provide 8.16 and I think quite a number of servers run plain Ubuntu LTS.

@mvz
Copy link

mvz commented Dec 22, 2019

Also, Travis uses Xenial which comes with node 8.12.0.

jbielick added a commit to jbielick/webpacker that referenced this pull request Feb 10, 2020
8.16 is somewhat arbitrary as far as I can tell, so this allows earlier 8.x releases of node.js as the engine so that 8.9, 8.10, 8.12, etc from Ubuntu 18.04 LTS can install this package.

rails#2228
jakeNiemiec added a commit that referenced this pull request Feb 10, 2020
#2454)

* Allow node.js 8.9 engine compatibility

8.16 is somewhat arbitrary as far as I can tell, so this allows earlier 8.x releases of node.js as the engine so that 8.9, 8.10, 8.12, etc from Ubuntu 18.04 LTS can install this package.

#2228

* restrict to compatible yarn engine versions

Co-Authored-By: Jake Niemiec <[email protected]>

* restrict to compatible node engine versions

As 13 has been released, an upper limit is prudent as well.

Co-Authored-By: Jake Niemiec <[email protected]>

Co-authored-by: Jake Niemiec <[email protected]>
KingTiger001 added a commit to KingTiger001/Rails-web-pack-project that referenced this pull request Jan 15, 2023
…) (#2454)

* Allow node.js 8.9 engine compatibility

8.16 is somewhat arbitrary as far as I can tell, so this allows earlier 8.x releases of node.js as the engine so that 8.9, 8.10, 8.12, etc from Ubuntu 18.04 LTS can install this package.

rails/webpacker#2228

* restrict to compatible yarn engine versions

Co-Authored-By: Jake Niemiec <[email protected]>

* restrict to compatible node engine versions

As 13 has been released, an upper limit is prudent as well.

Co-Authored-By: Jake Niemiec <[email protected]>

Co-authored-by: Jake Niemiec <[email protected]>
smartech7 pushed a commit to smartech7/ruby-webpacker that referenced this pull request Aug 4, 2023
…) (#2454)

* Allow node.js 8.9 engine compatibility

8.16 is somewhat arbitrary as far as I can tell, so this allows earlier 8.x releases of node.js as the engine so that 8.9, 8.10, 8.12, etc from Ubuntu 18.04 LTS can install this package.

rails/webpacker#2228

* restrict to compatible yarn engine versions

Co-Authored-By: Jake Niemiec <[email protected]>

* restrict to compatible node engine versions

As 13 has been released, an upper limit is prudent as well.

Co-Authored-By: Jake Niemiec <[email protected]>

Co-authored-by: Jake Niemiec <[email protected]>
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.

4 participants