[sdk-testgen] bump @azure-tools/codegen to 2.10.1#14334
Open
mikeharder wants to merge 1 commit intomainfrom
Open
[sdk-testgen] bump @azure-tools/codegen to 2.10.1#14334mikeharder wants to merge 1 commit intomainfrom
mikeharder wants to merge 1 commit intomainfrom
Conversation
- 2.10.1 is actually newer than 3.0.283, which was a preview
Contributor
There was a problem hiding this comment.
Pull request overview
This PR aligns the @azure-tools/codegen dependency in tools/sdk-testgen to the stable 2.10.1 version, removing the previously specified ^3.0.283 which was a preview series. Both @autorest/codemodel and @autorest/extension-base (the main upstream dependencies) already required ~2.10.0 of codegen, so the old 3.x declaration was installing an extra, unnecessary top-level copy that has now been eliminated.
Changes:
package.json: Downgrades@azure-tools/codegenspec from^3.0.283to^2.10.1(stable vs. preview).package-lock.json: Removes the now-unnecessary@azure-tools/codegen@3.0.283top-level entry and its exclusive dependencies (@azure-tools/linq,@azure-tools/async-io@4.0.246,@azure-tools/tasks@4.0.246, oldjs-yaml@3.13.1,semver@5.x), hoisting the shared2.10.1packages to the top-level; also updatesjs-yaml4.x to4.1.1and 3.x to3.14.2, and hoistsretryto the top level.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
tools/sdk-testgen/package.json |
Updates @azure-tools/codegen version constraint from ^3.0.283 to ^2.10.1 |
tools/sdk-testgen/package-lock.json |
Removes 3.x codegen and its exclusive transitive deps; consolidates 2.10.1 packages at the top level; updates js-yaml minor patch versions; hoists retry to top level |
Files not reviewed (1)
- tools/sdk-testgen/package-lock.json: Language not supported
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.