Juntuchen/alpha4 ops #32354
Draft
Juntuchen/alpha4 ops #32354
Conversation
This file contains 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
with `rush add --dev -m -p typescript@~5.7.2` except `dev-tool` due to copmilation error detailed at #32278. Also bump versions in non-rush projects and samples. ***NO_CI*** cosmos: fix new error about implicit `any[]` storage-blob: react to `Buffer` typing change storage-file-datalake test: react to Uint8Array typing change
…32283) ### Packages impacted by this PR - @azure/dev-tool - @azure/arm-advisor - @azure/arm-agrifood - @azure/arm-apicenter ### Issues associated with this PR - #31338 ### Describe the problem that is addressed by this PR Migrates @azure/arm-advisor to ESM/vitest via @azure/dev-tool and adds fixes such as `vi.fn()` support and `import "dotenv/config";` Also adds support for ESM testing and type testing. ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary) --------- Co-authored-by: Jeremy Meng <[email protected]> Co-authored-by: Deyaaeldeen Almahallawi <[email protected]>
and replace usage with `tar` and `unzipper`. `decompress` pulls in `decompress-tar` which has a vulnerability https://security.snyk.io/package/npm/decompress-tar. However, `decompress-tar` hasn't been updated for seven years, and there's no sign of this issue being addressed in the future. This PR switches to use more actively maintained packages `tar` and `unzipper` to extract from compressed archives and removes `decompress` dependency.
### Packages impacted by this PR - @azure/dev-tool ### Issues associated with this PR ### Describe the problem that is addressed by this PR Fixes the source migration to not look for the `NodeNext` but to make the migration. Also fixes the idempotency of the migration for config files. ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
- some eslint-disable directives no long apply - dev-tool stays at typescript@~5.6.2 but its unit-test baseline got upgraded incorrectly in the last NO_CI commit
### Packages impacted by this PR @azure/monitor-opentelemetry-exporter ### Issues associated with this PR #32205 ### Describe the problem that is addressed by this PR Execute Samples failing in nightly runs ### Checklists - [x] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [x] Added a changelog (if necessary)
- Extract several helpers into helpers.js so that it is easier to add tests for them. - update jsdoc to support TypeScript checks - add some tests of expetations (currently failing)
### Packages impacted by this PR communication-sms ### Issues associated with this PR ### Describe the problem that is addressed by this PR Fix the error in optOut.remove method ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
### Packages impacted by this PR - @azure-tools/test-utils-vitest ### Issues associated with this PR ### Describe the problem that is addressed by this PR Adds multi-version testing to vitest, which requires globals in order to work. ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary) --------- Co-authored-by: Jeremy Meng <[email protected]>
After upgrade to TS 5.7.2.
### Packages impacted by this PR New package @azure/ai-projects ### Issues associated with this PR ### Describe the problem that is addressed by this PR AI Projects with Agent capabilities more details here https://learn.microsoft.com/en-us/azure/ai-studio/how-to/create-projects?tabs=ai-studio https://learn.microsoft.com/en-us/azure/ai-studio/how-to/develop/sdk-overview ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [x ] Added impacted package name to the issue description - [ x] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [x ] Added a changelog (if necessary) --------- Co-authored-by: Grace Brigham <[email protected]> Co-authored-by: Grace Brigham <[email protected]> Co-authored-by: ZachhK <[email protected]> Co-authored-by: Zachary King <[email protected]> Co-authored-by: ansaxena1 <[email protected]> Co-authored-by: Jeremy Meng <[email protected]> Co-authored-by: Jeff Fisher <[email protected]>
The custom rules to suppress errors/warnings for bin scripts were removed in previous commits. This PR restores them.
### Packages impacted by this PR None directly, this only touches CI. ### Describe the problem that is addressed by this PR We were seeing odd behavior in CI where the `rush install` inside of the lint step was failing, but since this task was only ever called from analyze, which has already executed `rush install` it seemed unnecessary. This change inlines the behavior from the run-estlint step and avoids the duplicate `rush install` call.
### Packages impacted by this PR N/A ### Issues associated with this PR N/A ### Describe the problem that is addressed by this PR The "Inline Images" example was missing from the most recent release of our SDK. ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? N/A ### Are there test cases added in this PR? _(If not, why?)_ N/A - readme only ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
github-actions
bot
added
App Configuration
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.
Packages impacted by this PR
Issues associated with this PR
Describe the problem that is addressed by this PR
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists