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

chore(gatsby-cli): Add notice for Node 8 EOL #20466

Merged
merged 5 commits into from
Jan 9, 2020

Conversation

sidharthachatterjee
Copy link
Contributor

@sidharthachatterjee sidharthachatterjee commented Jan 8, 2020

Now that Node 8 has hit EOL, we will stop actively supporting it and raise our minimum supported Node version to the next LTS (10.13) as per support policy.

However, we won't stop support immediately because we currently have roughly 6% of our users using Node < 10.13 (based on data over the past month).

Screenshot 2020-01-06 at 3 13 45 PM

This PR adds a deprecation warning for now and should set the stage for dropping support in a month from now.

@sidharthachatterjee sidharthachatterjee requested a review from a team as a code owner January 8, 2020 08:25
@sidharthachatterjee sidharthachatterjee added the type: maintenance An issue or pull request describing a change that isn't a bug, feature or documentation change label Jan 8, 2020
packages/gatsby-cli/src/index.ts Outdated Show resolved Hide resolved
freiksenet
freiksenet previously approved these changes Jan 8, 2020
@sidharthachatterjee sidharthachatterjee added the bot: merge on green Gatsbot will merge these PRs automatically when all tests passes label Jan 8, 2020
@sidharthachatterjee sidharthachatterjee requested a review from a team as a code owner January 8, 2020 12:45
@gatsbybot gatsbybot merged commit a5f1284 into master Jan 9, 2020
@delete-merged-branch delete-merged-branch bot deleted the chore/deprecation-warning-for-node-8 branch January 9, 2020 08:59
report.warn(
report.stripIndent(`
Node.js ${process.version} has reached End of Life status on 31 December, 2019.
Gatsby will only actively support ${NEXT_MIN_NODE_VERSION} or higher and drop support for Node 8 soon.
Copy link
Contributor

@muescha muescha Jan 9, 2020

Choose a reason for hiding this comment

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

maybe remove here the version number so on the next time we don't need to change the numbers...

Suggested change
Gatsby will only actively support ${NEXT_MIN_NODE_VERSION} or higher and drop support for Node 8 soon.
Gatsby will only actively support ${NEXT_MIN_NODE_VERSION} or higher and drop support for earlier Node versions soon.

if ok i can create a follow up PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: merge on green Gatsbot will merge these PRs automatically when all tests passes type: maintenance An issue or pull request describing a change that isn't a bug, feature or documentation change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants