Skip to content

v3 compatibility experiment: uuid 8.3.2 + Node 16/18 matrix#449

Closed
TsuyoshiUshio wants to merge 37 commits into
v4.xfrom
tsushi/v3-uuid-8-3-2-node16-18
Closed

v3 compatibility experiment: uuid 8.3.2 + Node 16/18 matrix#449
TsuyoshiUshio wants to merge 37 commits into
v4.xfrom
tsushi/v3-uuid-8-3-2-node16-18

Conversation

@TsuyoshiUshio

Copy link
Copy Markdown
Contributor

Summary

  • pin uuid from ^13.0.2 to 8.3.2 for CommonJS compatibility experiments
  • add Node 16 and Node 18 to azure-pipelines/templates/test.yml matrix
  • keep existing Node 20/22/24 jobs

Validation

  • npm test (pass locally)
  • Node 16 require smoke: require('uuid') works
  • Node 18 require smoke: require('uuid') works

Security Note

  • npm audit --omit=dev reports GHSA-w5hq-g745-h8pq on uuid < 11.1.1 (moderate)
  • this PR is intended to validate compatibility/CI behavior, including whether security gates block this change

ejizba and others added 30 commits August 18, 2022 12:17
And replicate old host behavior to prevent breaking changes
* fix bug

* add validation

* remove unnecessary ts flags
* initial try

* add testing

* pass context

* rename

* remove suppression of extra log warning

* rename and suppress only the one error

* remove test file
Bumps [webpack](https://github.com/webpack/webpack) from 5.74.0 to 5.76.0.
- [Release notes](https://github.com/webpack/webpack/releases)
- [Commits](webpack/webpack@v5.74.0...v5.76.0)

---
updated-dependencies:
- dependency-name: webpack
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [word-wrap](https://github.com/jonschlinkert/word-wrap) from 1.2.3 to 1.2.4.
- [Release notes](https://github.com/jonschlinkert/word-wrap/releases)
- [Commits](jonschlinkert/word-wrap@1.2.3...1.2.4)

---
updated-dependencies:
- dependency-name: word-wrap
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
ejizba and others added 7 commits June 12, 2024 10:59
Based on this PR from host: Azure/azure-functions-host#10256

And clean up CodeQL settings based on this PR: Azure/azure-functions-host#10198
Co-authored-by: Victoria Hall <victoria.hall@microsoft.com>
Bump the v3 package to 3.5.2 and update the uuid dependency to 13.0.2. Because uuid 13 is ESM-only for older Node.js CommonJS require paths, narrow the unit test matrix to Node 20, 22, and 24.

Co-authored-by: Tsuyoshi Ushio <tsushi@microsoft.com>
Co-authored-by: Dobby <dobby@microsoft.com>
@TsuyoshiUshio TsuyoshiUshio requested a review from a team as a code owner June 17, 2026 22:18
@TsuyoshiUshio

Copy link
Copy Markdown
Contributor Author

Closing in favor of #450, which targets v3.x as intended.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants