Skip to content

Conversation

@bobisjan
Copy link
Contributor

This removes remaining leftovers of legacy Node.js support code #13.

@bobisjan bobisjan force-pushed the node-cleanup branch 2 times, most recently from 398691e to 9859a54 Compare April 12, 2020 07:00
"express": "^4.13.3",
"fastboot": "^2.0.0",
"fastboot-express-middleware": "^2.0.0",
"fs-promise": "^2.0.3",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not used anymore.

"fastboot": "^2.0.0",
"fastboot-express-middleware": "^2.0.0",
"fs-promise": "^2.0.3",
"request": "^2.81.0"
Copy link
Contributor Author

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.

@bobisjan bobisjan changed the title Remove legacy Node.js support Remove legacy Node.js support leftovers Apr 12, 2020
@bobisjan
Copy link
Contributor Author

@rwjblue, would you take a look?

@bobisjan bobisjan mentioned this pull request Oct 24, 2020
1 task
## Requirements

FastBoot App Server requires Node.js v4.2 or later.
FastBoot App Server requires Node.js v8 or later.
Copy link
Member

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...

Copy link
Contributor Author

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.

@rwjblue rwjblue merged commit d4a75a7 into ember-fastboot:master Oct 24, 2020
@bobisjan bobisjan deleted the node-cleanup branch October 24, 2020 15:32
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.

2 participants