-
Notifications
You must be signed in to change notification settings - Fork 552
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
feat: drop node 6 support by no longer testing for it #1042
Conversation
62775e4
to
7234149
Compare
7234149
to
6f6ec60
Compare
@@ -2,10 +2,9 @@ | |||
"compilerOptions": { | |||
"outDir": "./dist", | |||
"pretty": true, | |||
"target": "es2015", | |||
"target": "es2017", |
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.
2020 right there
6f6ec60
to
ac783f0
Compare
package.json
Outdated
@@ -105,11 +105,11 @@ | |||
"@types/diff": "^3.5.2", | |||
"@types/lodash": "^4.14.136", | |||
"@types/needle": "^2.0.4", | |||
"@types/node": "^6.14.4", | |||
"@types/node": "13.9.2", |
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 you want to bump it to 13.x. Cos this will bring you methods that are available in Node 13. But our lowest support now is Node 8, so preferably this should be 8.10.59
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.
updated
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.
updated
ac783f0
to
98231d9
Compare
98231d9
to
566a54b
Compare
🎉 This PR is included in version 1.301.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
What does this PR do?
Stop supporting Node 6