Skip to content
This repository was archived by the owner on Nov 29, 2022. It is now read-only.

build: use node 14#5047

Closed
vhashimotoo wants to merge 1 commit intoelectron:masterfrom
vhashimotoo:vh/node-14
Closed

build: use node 14#5047
vhashimotoo wants to merge 1 commit intoelectron:masterfrom
vhashimotoo:vh/node-14

Conversation

@vhashimotoo
Copy link
Contributor

Node 14 is LTS and everything "should" be good.

Why you shouldn't trust me?

Mostly because node-sass can break everything. node-sass is used in the two exemplars, one from node-sass@5.0.0 and one from node-sass-middleware->node-sass@4.3.0 and factically this means we should download two prebuild binaries. Theoretically and factically, this dependency is fixed by #4765.

So I have prepared you, and on this, I end.

molant pushed a commit that referenced this pull request Mar 30, 2021
@molant molant mentioned this pull request Mar 30, 2021
@molant molant closed this in #5274 Mar 30, 2021
molant pushed a commit that referenced this pull request Mar 30, 2021
@molant
Copy link
Contributor

molant commented Mar 30, 2021

Closed via #5274

Thanks @vhashimotoo !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants