-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Debump Node.js requirement to 10.14.2 #3854
Conversation
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.
Can you verify our documentation is up to date? I feel like we added recently this info in a doc.
I searched with multiple keywords like |
Ah okay. Just checked |
|
Pulling this branch and testing locally for sanity check [Edit] Okay it took a while, but I was able to get tests passing locally. ADO just hates me. I'll keep babysitting. |
* Debump Node.js requirement to 10.14.2 * Update entry * Typo Co-authored-by: Corina <14900841+corinagum@users.noreply.github.com>
Changelog Entry
Fixed
>= 10.14.2
from>= 12.0.0
, by @compulim in PR #3854Description
De-bump Node.js engine requirement for Direct Line Speech SDK. Based on data from its
package-lock.json
, the minimal version supported is10.14.2
.Specific Changes
packages/directlinespeech/package.json
to setengines.node
to>= 10.14.2
I have added tests and executed them locallyCHANGELOG.md
I have updated documentationReview Checklist
z-index
)package.json
andpackage-lock.json
reviewed