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

[core] Drop node 6 support #14379

Merged
merged 3 commits into from
Feb 2, 2019
Merged

Conversation

eps1lon
Copy link
Member

@eps1lon eps1lon commented Feb 1, 2019

Node 6 will reach end of life in April 2019. The upcoming v4 provides a good opportunity to drop support for it (although a little bit to early depending on our release date).

Testing node 8 in CI which helps testing node 8 support. We never actually ran our tests with node 6. Might not work due to previous issues encountered in #13818.

Part of #13663

Breaking change

Require node >= 8 (was 6 previously)

@eps1lon eps1lon added this to the v4 milestone Feb 1, 2019
@eps1lon eps1lon mentioned this pull request Feb 1, 2019
56 tasks
@@ -14,7 +14,7 @@ Because Googlebot uses a web rendering service (WRS) to index the page content,

## Server

Because Material-UI supports server-side rendering, we need to support the latest, stable releases of [Node.js](https://github.com/nodejs/node). We try to support the [last active LTS version](https://github.com/nodejs/Release#lts-schedule1). Right now, we support **node v6.x** and newer versions.
Because Material-UI supports server-side rendering, we need to support the latest, stable releases of [Node.js](https://github.com/nodejs/node). We try to support the [last active LTS version](https://github.com/nodejs/Release#lts-schedule1). Right now, we support **node v8.x** and newer versions.
Copy link
Member Author

Choose a reason for hiding this comment

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

Question about that statement: Shouldn't it always have been "oldest LTS version"? The "latest active LTS version" would be node 10 now.

@oliviertassinari oliviertassinari merged commit 012747e into mui:next Feb 2, 2019
@eps1lon eps1lon deleted the breaking/drop-node-6 branch February 2, 2019 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants