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: drop Node6 support #4654
Conversation
Node6 was end-of-life'ed in the end of April, 2019, with AWS Lambda prohibiting updaets to the node6 runtime since June 30, 2019. This makes it quite safe for us to remove the Node 6 support from the repository. From now on, all the subsequent commits will require Node 8.9.0+.
I think this should be a major change |
@@ -5,7 +5,7 @@ | |||
"main": "index.js", | |||
"repository": "github:GoogleChrome/puppeteer", | |||
"engines": { | |||
"node": ">=6.4.0" | |||
"node": ">=8.9.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As a practical concern, is there anything from 7 that would break us?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure. I picked 8 here since it's an LTS release with a clear lifetime.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok. Turns out this only produces a warning.
* @param {string} text | ||
* @return {string} | ||
*/ | ||
function transformAsyncFunctions(text) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RIP
The test: Puppeteer-Web should work over web socket |
@@ -57,7 +57,8 @@ See [puppeteer vs puppeteer-core](https://github.com/GoogleChrome/puppeteer/blob | |||
|
|||
### Usage | |||
|
|||
Note: Puppeteer requires at least Node v6.4.0, but the examples below use async/await which is only supported in Node v7.6.0 or greater. | |||
Note: Prior to v1.18.1, Puppeteer required at least Node v6.4.0. All subsequent versions rely on | |||
Node 8.9.0+. All examples below use async/await which is only supported in Node v7.6.0 or greater. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's codify the supported Node.js versions over time as well. We could say something like:
Puppeteer follows the latest [maintenance LTS](https://github.com/nodejs/Release#release-schedule) version of Node.
Moved to #5045. |
Node6 was end-of-life'ed in the end of April, 2019, with AWS Lambda
prohibiting updaets to the node6 runtime since June 30, 2019.
This makes it quite safe for us to remove the Node 6 support from the
repository.
From now on, all subsequent releases will require Node 8.9.0+.