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
Remove legacy Node.js support leftovers #120
Conversation
398691e
to
9859a54
Compare
@@ -30,16 +30,12 @@ | |||
"compression": "^1.6.2", | |||
"express": "^4.13.3", | |||
"fastboot": "^2.0.0", | |||
"fastboot-express-middleware": "^2.0.0", | |||
"fs-promise": "^2.0.3", |
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 used anymore.
@@ -30,16 +30,12 @@ | |||
"compression": "^1.6.2", | |||
"express": "^4.13.3", | |||
"fastboot": "^2.0.0", | |||
"fastboot-express-middleware": "^2.0.0", | |||
"fs-promise": "^2.0.3", | |||
"request": "^2.81.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.
This is required by request-promise
for testing only.
@rwjblue, would you take a look? |
@@ -27,7 +27,7 @@ particular, you can provide a custom: | |||
|
|||
## Requirements | |||
|
|||
FastBoot App Server requires Node.js v4.2 or later. | |||
FastBoot App Server requires Node.js v8 or later. |
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.
Haha, I'd like to bump this again too but seems good to do separately...
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.
Yes 😬, #123 is the next step.
This removes remaining leftovers of legacy Node.js support code #13.