Skip to content
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

[AutoPR @azure/arm-healthcareapis] Dev healthcareapis microsoft.healthcare apis 2021 11 01 changes #6129

Closed

Conversation

openapi-sdkautomation[bot]
Copy link

Create to sync Azure/azure-rest-api-specs#16108

Installation Instructions

You can install the package @azure/arm-healthcareapis of this PR using the following command:

curl -L "https://portal.azure-devex-tools.com/api/sdk-dl-pub?p=Azure/16108/azure-sdk-for-js/@azure_arm-healthcareapis/azure-arm-healthcareapis-1.1.1.tgz" -o azure-arm-healthcareapis-1.1.1.tgz
npm install azure-arm-healthcareapis-1.1.1.tgz

Direct Download

The generated package can be directly downloaded from here:

@openapi-sdkautomation openapi-sdkautomation bot force-pushed the sdkAuto/16108/@azure_arm-healthcareapis branch 2 times, most recently from 04d913d to 245f8b3 Compare September 21, 2021 23:44
@openapi-sdkautomation openapi-sdkautomation bot force-pushed the sdkAuto/16108/@azure_arm-healthcareapis branch from 245f8b3 to 5caf074 Compare October 25, 2021 20:09
@openapi-sdkautomation openapi-sdkautomation bot force-pushed the sdkAuto/16108/@azure_arm-healthcareapis branch 8 times, most recently from fa2f179 to ea0d26f Compare December 8, 2021 23:03
azure-sdk and others added 18 commits January 27, 2022 09:48
…zure#19804)

* initial backoff implementation

* added test for backoff

* improved the autorefresh documentation

* better explained scenarios

* clarify tokenRefresher's expected return type
**Checklists** 
- [x] Added impacted package name to the issue description

**Packages impacted by this PR:**
@azure-rest/agrifood-farming

**Describe the problem that is addressed by this PR:**
Re-generate Agrifood Farming using the latest version of RLC generator. This package was one of the first-ever RLCs generated and codegen has changes quite a bit since then. Regenerating against the same swagger. Major changes since last generation:

- Switched away from @azure-rest/core-client-lro and @azure-rest/core-client-paging in favor of @azure/core-lro and @azure/core-client
- Split Output and Input models



**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)_
NO

**Are there test cases added in this PR?**_(If not, why?)_
NO, it already has tests

**Provide a list of related PRs**_(if any)_
Azure#19897

**Command used to generate this PR:**_(Applicable only to SDK release request PRs)_
`rushx generate:client`
…e format (Azure#20091)

* [Schema Registry Avro] Support backward compatibility for old preamble format

* fix linting

* address feedback
…re#19954)

- Fixes Azure#19809
- Part of work towards Azure#18223

The main motivation of this PR was to add support for the new string sanitizers introduced in Azure/azure-sdk-tools#2530. As part of this, I've also tackled some refactoring that will be required for session-level sanitizer support (Azure#18223) where we will be wanting to enable adding sanitizers without access to an instance of the `Recorder` class. While implementing the new sanitizer logic, I refactored the `addSanitizers` method into smaller chunks to make adding additional sanitizers easier. To summarize the changes:

* Removed the `Sanitizer` class, instead making the `addSanitizers` function in `sanitizer.ts` take in a `HttpClient` and recording ID as parameter.
* Refactored the `addSanitizers` function to call smaller functions for each sanitizer (some of which are a bit FP-style) instead of using if statements + special cases. Hopefully this will make things a bit easier to maintain.
* Some other minor refactors (e.g. extracting duplicated `createRecordingRequest` function into a utility).
* Add support for the string sanitizers in what I think is the most logical way, but there is a **breaking change**:
  * When calling `addSanitizers`, instead of specifying `generalRegexSanitizers: [...]` etc., you now specify `generalSanitizers: [...]`. Both regex sanitizers and string sanitizers can be used in this way, for example:
 ```ts
recorder.addSanitizers({
  generalSanitizers: [
    {
      regex: true, // Regex matching is enabled by setting the 'regex' option to true.
      target: ".*regex",
      value: "sanitized",
    },
    {
      // Note that `regex` defaults to false and doesn't need to be specified when working with bare strings.
      // In my experience, this is the most common scenario anyway.
      target: "Not a regex",
      value: "sanitized",
    }
  ],
});
```
### Packages impacted by this PR
@azure/keyvault-keys

### Issues associated with this PR
Resolves Azure#19857

### Describe the problem that is addressed by this PR
Now that this flag has been added we can regenerate code and add support for
immutable key release policies. Once a policy is marked immutable it can no
longer be modified.

### Are there test cases added in this PR? _(If not, why?)_
Yes, I debated whether I should also add a test case for "create a key, then
update the release policy to be immutable, then try to change it" but it
wouldn't drive any code so I skipped it.

Do note that since MHSM does not currently support this flag I was unable to add
a test for it for MHSM, but I will create an issue to add that in later.
…ointstore-table``` (Azure#20106)

* enforce sort-imports

* prettier
### Packages impacted by this PR
@azure/core-tracing

### Issues associated with this PR
N/A - created after discussion with some team members

### Describe the problem that is addressed by this PR
`withSpan` is meant to await some user-provided callback and set the appropriate
status on the created span before closing it. Because we have no way of knowing
whether the callback is sync or async we must promisify it first. Up until now that
meant that the return type of a promise-returning callback was
`Promise<Promise<...>>` which does not reflect the reality that `await` will
recursively unwrap promises. That meant that the consumer must also either
`await` or mark the function as `async` in order to avoid a return-type
mismatch.

### 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?
The solution here is to introduce an `Resolved` type which is a narrower
version of TypeScript 4.5's `Awaited` type. Because our min-bar is lower we
cannot take advantage of this new type yet, so I introduced something like it
which can be replaced with the broader type once we support TS >= 4.5.
* [Schema Registry Avro] Update cache policy

* renames

* use lru-cache

* add tests

* update changelog

* update test

* update test

* update semver for @types/lru-cache
commit dcb5df3
Merge: 356a32c 6739271
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 28 17:22:23 2022 -0500

    Merge remote-tracking branch 'upstream/main' into witemple-msft/rollup2

commit 356a32c
Merge: 6527b28 b88c0ba
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 28 16:51:32 2022 -0500

    Merge remote-tracking branch 'upstream/main' into witemple-msft/rollup2

commit 6527b28
Merge: db7197b 55ad309
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 28 16:13:56 2022 -0500

    Merge remote-tracking branch 'upstream/main' into witemple-msft/rollup2

commit db7197b
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 28 11:03:50 2022 -0500

    Enable communication-phone-numbers browser tests

commit 9336dc1
Merge: 097c9d2 7ea04a8
Author: Will Temple <witemple@microsoft.com>
Date:   Thu Jan 27 16:17:54 2022 -0500

    Merge branch 'witemple-msft/rollup2' of github.com:Azure/azure-sdk-for-js into witemple-msft/rollup2

commit 097c9d2
Merge: a022f51 9ef90e4
Author: Will Temple <witemple@microsoft.com>
Date:   Thu Jan 27 16:17:29 2022 -0500

    Merge remote-tracking branch 'upstream/main' into witemple-msft/rollup2

commit 7ea04a8
Author: Will Temple <witemple@microsoft.com>
Date:   Wed Jan 26 17:00:14 2022 -0500

    Update common/tools/dev-tool/src/config/rollup.base.config.ts

    Co-authored-by: Harsha Nalluru <sanallur@microsoft.com>

commit a022f51
Merge: 64599af c702456
Author: Will Temple <witemple@microsoft.com>
Date:   Wed Jan 26 16:42:10 2022 -0500

    Merge remote-tracking branch 'upstream/main' into witemple-msft/rollup2

commit 64599af
Merge: bdd4bca 0c4d2af
Author: Will Temple <witemple@microsoft.com>
Date:   Tue Jan 25 17:29:58 2022 -0500

    Merge remote-tracking branch 'upstream/main' into witemple-msft/rollup2

commit bdd4bca
Merge: e834674 97b77df
Author: Will Temple <witemple@microsoft.com>
Date:   Tue Jan 25 11:26:58 2022 -0800

    Merge remote-tracking branch 'upstream/main' into witemple-msft/rollup2

commit e834674
Author: Will Temple <witemple@microsoft.com>
Date:   Tue Jan 25 11:19:29 2022 -0800

    fixed schema-registry-avro

commit 26425ec
Author: Will Temple <witemple@microsoft.com>
Date:   Mon Jan 24 12:53:58 2022 -0800

    Remove util dependency from monitor-query

commit 50beb01
Author: Will Temple <witemple@microsoft.com>
Date:   Mon Jan 24 14:36:30 2022 -0500

    Removed dangling 'util' dependency

commit d28baf0
Author: Will Temple <witemple@microsoft.com>
Date:   Mon Jan 24 12:38:56 2022 -0500

    Make warning inhibitors work on Windows

commit 5c4c828
Author: Will Temple <witemple@microsoft.com>
Date:   Mon Jan 24 08:31:07 2022 -0800

    Fixed recorder build:test

commit 81a9d6d
Merge: cc7f212 2144ad4
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 21 15:16:24 2022 -0800

    Merge remote-tracking branch 'upstream/main' into witemple-msft/rollup2

commit cc7f212
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 21 15:16:08 2022 -0800

    iot-device-update-rest: migrated

commit 88212d2
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 21 14:13:51 2022 -0800

    keyvault-keys: disable node polyfill

commit 3c1302b
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 21 14:10:17 2022 -0800

    keyvault-admin: disabled browser node polyfill

commit 818cf06
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 21 13:35:29 2022 -0800

    Add basePath configuration to shared script config, inhibit empty warnings from node-resolve

commit 8d18789
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 21 12:15:58 2022 -0800

    Removed errant rollup.config.js entries

commit d4833d4
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 21 11:47:45 2022 -0800

    Remove my launch config

commit e63d4e5
Merge: 37cb4bc d9fe264
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 21 11:45:31 2022 -0800

    Merge remote-tracking branch 'upstream/main' into witemple-msft/rollup2

commit 37cb4bc
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 21 11:44:20 2022 -0800

    Added shim plugin for source maps

commit 98579cc
Merge: 9132065 e3db6c4
Author: Will Temple <witemple@microsoft.com>
Date:   Thu Jan 20 09:44:42 2022 -0800

    Merge remote-tracking branch 'upstream/main' into witemple-msft/rollup2

commit 9132065
Author: Will Temple <witemple@microsoft.com>
Date:   Fri Jan 14 14:39:32 2022 -0500

    Changed weird regex to path.split.join

commit 9c20ebb
Author: Will Temple <witemple@microsoft.com>
Date:   Thu Jan 13 17:25:21 2022 -0500

    Removed communication-chat browser test config, as it's not needed

commit 2a8364d
Author: Will Temple <witemple@microsoft.com>
Date:   Thu Jan 13 17:06:02 2022 -0500

    Deleted rollup.config.js for packages on the shared script.

commit 3fc9575
Author: Will Temple <witemple@microsoft.com>
Date:   Thu Jan 13 17:01:31 2022 -0500

    template: removed rollup

commit 85d7050
Author: Will Temple <witemple@microsoft.com>
Date:   Thu Jan 13 16:54:55 2022 -0500

    synapse: silence rollup output

commit ae9f7a6
Author: Will Temple <witemple@microsoft.com>
Date:   Thu Jan 13 16:30:53 2022 -0500

    Some updates to the bundle command

commit 11cee51
Author: Will Temple <witemple@microsoft.com>
Date:   Thu Jan 13 16:30:33 2022 -0500

    Migrate more packages to shared rollup script

commit 0c4d89c
Merge: f1ccb03 78f849d
Author: Will Temple <witemple@microsoft.com>
Date:   Thu Jan 13 13:00:13 2022 -0500

    Merge remote-tracking branch 'upstream/main' into dev-tool/rollup2

commit f1ccb03
Author: Will Temple <witemple@microsoft.com>
Date:   Thu Jan 13 12:11:27 2022 -0500

    [dev-tool] Add rollup-plugin-polyfill-node

commit 1a8ec9b
Author: Will Temple <witemple@microsoft.com>
Date:   Wed Jan 12 19:07:43 2022 -0500

    preferBuiltins: false for browser

commit fe81f4c
Author: Will Temple <witemple@microsoft.com>
Date:   Wed Jan 12 18:20:38 2022 -0500

    arm-compute: fix new warning

commit a75eca0
Author: Will Temple <witemple@microsoft.com>
Date:   Wed Jan 12 16:20:54 2022 -0500

    Fixed build error in dev-tool

commit d890a89
Author: Will Temple <witemple@microsoft.com>
Date:   Wed Jan 12 16:18:36 2022 -0500

    Resoved merge conflict

commit a64fa41
Author: Will Temple <witemple@microsoft.com>
Date:   Wed Jan 12 16:12:16 2022 -0500

    Remove mixed rollup commands, leaving only rollup.test.config.js

commit f159571
Author: Will Temple <witemple@microsoft.com>
Date:   Wed Jan 12 14:43:55 2022 -0500

    Remove all rollup dependencies

commit 6517014
Author: Will Temple <witemple@microsoft.com>
Date:   Wed Jan 12 14:11:28 2022 -0500

    Remove ordinary rollup commands.

commit f43cc70
Author: Will Temple <witemple@microsoft.com>
Date:   Wed Jan 12 12:48:19 2022 -0500

    Removed configs that opt-out of browser bundles

commit 24357be
Merge: 86250f0 20df85c
Author: Will Temple <witemple@microsoft.com>
Date:   Wed Jan 12 12:31:49 2022 -0500

    Merge remote-tracking branch 'upstream/main' into dev-tool/rollup2

commit 86250f0
Merge: 26c241e 8dcc094
Author: Will Temple <witemple@microsoft.com>
Date:   Wed Jan 12 12:27:43 2022 -0500

    Merge remote-tracking branch 'upstream/main' into dev-tool/rollup2

commit 26c241e
Merge: a0a98ee 12b1941
Author: Will Temple <witemple@microsoft.com>
Date:   Tue Dec 14 18:37:30 2021 -0500

    Merge remote-tracking branch 'upstream/main' into dev-tool/rollup2

commit a0a98ee
Author: Will Temple <witemple@microsoft.com>
Date:   Tue Dec 14 17:39:12 2021 -0500

    Make consistent

commit c8f6ffe
Author: Will Temple <witemple@microsoft.com>
Date:   Tue Dec 14 17:29:05 2021 -0500

    Migrate template

commit d56bcc5
Author: Will Temple <witemple@microsoft.com>
Date:   Wed Dec 8 11:46:16 2021 -0500

    [dev-tool] Add "bundle" command
…ure#20125)

* updates to tests and package.json

* delete old recordings

* add recordings - first pass

* fix setup

* TestProxy: true

* karma fixes

* address Jose's feedback

* formatting
timovv and others added 28 commits February 19, 2022 09:22
### Packages impacted by this PR

- `@azure/ai-text-analytics`

### Issues associated with this PR

- Azure#19859

### Describe the problem that is addressed by this PR

Migrates Text Analytics to new recorder.
The focus of these changes is to add the Fabric Bot rules into the repository.
* dtdl-parser updates

* add spelling fixes

* update release date
* datafactory-track2

* update

* update

* update

* update
* dnsresolver-release

* update

* update

* update
Post release automated changes for azure-arm-dnsresolver
Post release automated changes for azure-arm-datafactory
Post release automated changes for azure-arm-cosmosdb
Packages impacted by this PR
None. This PR adds documentation for Pipeline and PipelinePolicy

Issues associated with this PR
Azure#13561

Describe the problem that is addressed by this PR
Add documentation for the Pipeline and Policies and how to customize them.
* containerservice-track2-relesae

* update
* migrated clientArguments, removed circular deps

* removed the dependency on @azure/core-http

* policy converted to core rest pipeline

* removed the old policy, bumped major version

* added a changelog entry

* added test

* updated pnpm file
Co-authored-by: praveenkuttappan <prmarott@microsoft.com>
* Deprecate 1.0.0 version

* Update version to latest
* Fix up parameters

* Update general azcopy tool for copying from blobstorage

Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
* Update Dependency Versions

* Update version
…0553)

Post release automated changes for azure-arm-containerservice
### Packages impacted by this PR
@azure/opentelemetry-instrumentation-azure-sdk

### Issues associated with this PR
Resolves Azure#18738

### Describe the problem that is addressed by this PR
This PR solves the following:
1. I would like to have an end-to-end test that uses our pipeline and ensures
tracing works as expected. The idea here is to ensure we do not regress in the
future when making core changes. I added a node-only integration test that uses
core-rest-pipeline and a fake client.
1. We currently maintain a TestTracer, TestTracerProvider, and TestSpan
independently in multiple places. OpenTelemetry already provides testing utils
in their base package. I replaced our Test* objects with an InMemorySpanExporter
which allows us to grab all exported spans and inspect them.
1. In-passing, I realized that for a bit there could be a drift between client
packages that use the new naming conventions and core packages that use the old
naming conventions to pull az.namespace into a span. While it doesn't feel
appropriate to add compatibility code in our core packages, this package will be
in beta for quite some time so I think it's fine to add it here temporarily.
Merge 48158491575e1214f7f915326da78117a76e95d9 into 3f78f3bc2d2546511853ad1931360ccdd34b0175
@openapi-sdkautomation openapi-sdkautomation bot force-pushed the sdkAuto/16108/@azure_arm-healthcareapis branch from ea0d26f to 7c09960 Compare February 28, 2022 19:34
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.