Skip to content

Conversation

zzvswxy
Copy link
Owner

@zzvswxy zzvswxy commented Jun 29, 2022

No description provided.

deyaaeldeen and others added 30 commits May 16, 2022 22:09
* [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
in public-trimmed dts rollup, @beta APIs are removed. Because we only ship
public-trimmed version it prevents us from gathering feedback for @beta APIs.
This PR enable generation of beta-trimmed dts rollup.

Use beta-trimmed dts rollup for beta release
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
…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.
HarshaNalluru and others added 30 commits June 23, 2022 23:04
…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]>
)

* Add OpenTelemetry Metrics Exporter

* Format

* Addressing comments

* Addressing comments
* 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.