Skip to content
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 Speech SDK to 1.17.0 #4533

Merged
merged 3 commits into from
Nov 29, 2022
Merged

Conversation

compulim
Copy link
Contributor

@compulim compulim commented Nov 29, 2022

Fixes #4523.

Changelog Entry

Changed

Description

In some recent versions of microsoft-cognitiveservices-speech-sdk (a.k.a. Speech SDK), they are having some issues with non-npm package manager, such as yarn or pnpm.

They will fix the issue in January 2023 release. However, to unblock our current customers, we need to debump it back to a known good version (1.17.0).

When we perform manual release testing, we need to make sure Web Chat works with yarn and/or pnpm.

Design

microsoft-cognitiveservices-speech-sdk/package.json has a preinstall script which only works with npm but not yarn or pnpm. They are using this special script for npm-force-resolutions.

In the next release of Speech SDK, they will fix the issue by two changes:

Today, to unblock our customers, we need to debump Speech SDK back to 1.17.0.

Specific Changes

  • Run npm install --save-exact microsoft-cognitiveservices-speech-sdk@1.17.0 web-speech-cognitive-services@7.1.4 under packages/bundle and packages/directlinespeech
  • I have added tests and executed them locally
  • I have updated CHANGELOG.md
  • I have updated documentation

Review Checklist

This section is for contributors to review your work.

  • Accessibility reviewed (tab order, content readability, alt text, color contrast)
  • Browser and platform compatibilities reviewed
  • CSS styles reviewed (minimal rules, no z-index)
  • Documents reviewed (docs, samples, live demo)
  • Internationalization reviewed (strings, unit formatting)
  • package.json and package-lock.json reviewed
  • Security reviewed (no data URIs, check for nonce leak)
  • Tests reviewed (coverage, legitimacy)

@compulim compulim added this to the release-4.15.6 milestone Nov 29, 2022
@compulim compulim marked this pull request as ready for review November 29, 2022 18:23
@compulim compulim merged commit 42b07cc into microsoft:main Nov 29, 2022
@compulim compulim deleted the debump-speech-sdk branch November 29, 2022 20:15
beyackle2 pushed a commit that referenced this pull request Dec 1, 2022
* Debump Speech SDK to 1.17.0

* Update entry

* Debump Speech SDK 1.17.0
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.

NPM install may fail when using yarn or pnpm
2 participants