forked from Azure/azure-sdk-for-js
-
Notifications
You must be signed in to change notification settings - Fork 0
[AutoPR-agrifood]autogenerated #67
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
Open
zzvswxy
wants to merge
431
commits into
main
Choose a base branch
from
sdkGen/agrifood-js-367a579f-73b3-49c1-891e-c22f5cb71847/arm-agrifood-@azure/arm-agrifood
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[AutoPR-agrifood]autogenerated #67
zzvswxy
wants to merge
431
commits into
main
from
sdkGen/agrifood-js-367a579f-73b3-49c1-891e-c22f5cb71847/arm-agrifood-@azure/arm-agrifood
Conversation
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
* [core] Fix link in README * Update sdk/core/README.md Co-authored-by: Jeff Fisher <[email protected]> Co-authored-by: Jeff Fisher <[email protected]>
* Generated functionality for Operator Connect MVP0. * Reverting unwanted changes in arm-appservice * Updated SDK version. * Added entry to changelog. * Added new recordings * Fixing tests. * Fixing tests. * Fixing AAD test. * Modified recordings to address PR comments * Fixed version in changelog. * Fixing package version. * Revert "Fixing package version." This reverts commit cb6312b. * Added tag to beta as unreleased * Reapplying fix of version to alpha * Fixing package version.
- for eslint-plugin-azure-sdk and service-bus - remove unused `glob` dependencies from core-http
- eslint to ^8.0.0 - @types/eslint to ~8.4.0 - @typescript-eslint plugins to ~5.22.0 * Fix broken test because of class property node type changes. Improve test a bit to also verifying error line and column numbers * Disable @typescript-eslint/no-invalid-this After upgrading this rule reports more false positives. The following comment on the github issue suggests that it doesn't work for some callbacks because it is just a simple rule that doesn't do type analysis. typescript-eslint/typescript-eslint#3620 (comment) * Fix linter issues after upgrading the plugins in eslint-plugin-azure-sdk. Also upgrade eslint version in the impacted packages
* changes-track2-release * update
* servicelinker-track2-release * update * Update README.md No change required in readme.md Co-authored-by: Mary Gao <[email protected]>
…21893) * [core-lro] Set isCancelled when status is cancelled * don't check for isCanceled in TA test * fix lint * address feedback and handle cancellation uniformly * address feedback * add tests * edit * revert behavioral change * Update sdk/textanalytics/ai-text-analytics/package.json Co-authored-by: Will Temple <[email protected]>
* Update CHANGELOG.md * Update CHANGELOG.md
* containerservice-track2 * update
for the `omitMessageBody` feature.
Previous change update the package to include beta-trimmed dts rollup but missed updating the `types` field. This PR fixes it.
* [Text Analytics] Add migration guide * update link * update product label * address feedback * address feedback
Post release automated changes for azure-service-bus
…1923) Post release automated changes for azure-arm-containerservice
Post release automated changes for azure-arm-network
Post release automated changes for azure-arm-eventgrid
* appplatform-track2 * update * update
Post release automated changes for azure-arm-appplatform
…ackages ***NO_CI***
…zure#21948) Previously `npm view <packageName> versions --json` returns an array even when the result only contains one version. In latest NPM version in NodeJS v16 the result is now a string, which isn't expected by `semver` APIs. This PR wraps the single string result with an array.
…nMs` (Azure#22343) * Improves documentation for `maxAutoLockRenewalDurationInMs` * Update sdk/servicebus/service-bus/src/models.ts Co-authored-by: Jesse Squire <[email protected]> * Update sdk/servicebus/service-bus/src/models.ts Co-authored-by: Jesse Squire <[email protected]> * Update sdk/servicebus/service-bus/src/models.ts Co-authored-by: Jesse Squire <[email protected]> * Apply suggestions from code review * improve * PR LINK Co-authored-by: Jesse Squire <[email protected]>
Co-authored-by: ZiWei Chen (WICRESOFT NORTH AMERICA LTD) <[email protected]>
…e#22355) Post release automated changes for azure-arm-operationalinsights
* Support local addons path override in stress test deployment * Support username based deployId in local stress deployment * Support WhatIf in stress infrastructure provision script * Simplify stress user detection Co-authored-by: Wes Haggard <[email protected]> * Run helm plugin add with helper * Add WhatIf support to ps module install helper function Co-authored-by: Ben Broderick Phillips <[email protected]> Co-authored-by: Wes Haggard <[email protected]>
…ues (Azure#22345) * Adds support to run queries with group by over a column with null values * Update CHANGELOG.md * Prettifying the code * Update CHANGELOG.md * Update query.spec.ts
This is a holdover from when we used `createTracingFunction`. Now that this package is using TracingClient, the comment should reflect that.
### Packages impacted by this PR @azure/core-lro ### Issues associated with this PR N/A ### Describe the problem that is addressed by this PR The initial request method for an LRO could be a GET. See https://github.com/Azure/azure-rest-api-specs/blob/main/specification/web/resource-manager/Microsoft.Web/stable/2022-03-01/WebApps.json#L2981 ### 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?)_ Yes ### Provide a list of related PRs _(if any)_ N/A ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### 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)
…zure#22353) * release mediaservices package with package-account-2021-11 version * modify test file * update package.json and mediaservice_example.ts Co-authored-by: ZiWei Chen (WICRESOFT NORTH AMERICA LTD) <[email protected]>
* release compute package with package-2022-04-04 version * release compute package with package-2022-04-04 version * release compute package with package-2022-04-04 version Co-authored-by: ZiWei Chen (WICRESOFT NORTH AMERICA LTD) <[email protected]>
* release msi package with package-preview-2022-01 version * release msi package with package-preview-2022-01 version * release msi package with package-preview-2022-01 version Co-authored-by: ZiWei Chen (WICRESOFT NORTH AMERICA LTD) <[email protected]>
Post release automated changes for azure-arm-msi
Post release automated changes for azure-arm-compute
Post release automated changes for azure-arm-mediaservices
* delete valid links in ignore-links.txt * Update ignore-links.txt Co-authored-by: ZiWei Chen (WICRESOFT NORTH AMERICA LTD) <[email protected]>
Added myself to the CodeOwners for .github dir as I will be helping @jsquire with GitHub CXP on this repo.
From the tests point of view, the path of the initial request is inconsequential and could be anything. I updated the path property to be optional in `LroResponseSpec` and the implementation defaults it to "path". This simplifies the tests a bit because `path` now doesn't have to be specified in the first item in the routes list.
We had continueOnError set to true so if the emulator failed to start we would still try to run the tests which all fail but take a long time to run. The timeout for this step is about 10mins of trying so that should be enough time and so we also don't really need another retry step. Co-authored-by: Wes Haggard <[email protected]>
* Update verify-readme to take a single or multiple paths * chance ScanPaths to a comma delimited list and coalesce ScanPath/ScanPaths into a single variable for the script * VS Code was nice enough to add an extra single quote when adding a quote to the end of the line' * Capture ScanPaths.Split into an array so we don't have to call Split again Co-authored-by: James Suplizio <[email protected]>
upgrade cosmos with azure core tracing for cosmos diagnostics.
* refresh analysissercices package * upload yaml * Update ci.mgmt.yml Co-authored-by: ZiWei Chen (WICRESOFT NORTH AMERICA LTD) <[email protected]>
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.
No description provided.