forked from Azure/azure-sdk-for-js
-
Notifications
You must be signed in to change notification settings - Fork 1
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
openapi-sdkautomation
wants to merge
1,944
commits into
feature/v4
from
sdkAuto/16108/@azure_arm-healthcareapis
Closed
[AutoPR @azure/arm-healthcareapis] Dev healthcareapis microsoft.healthcare apis 2021 11 01 changes #6129
openapi-sdkautomation
wants to merge
1,944
commits into
feature/v4
from
sdkAuto/16108/@azure_arm-healthcareapis
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
Merged
25 tasks
openapi-sdkautomation
bot
force-pushed
the
sdkAuto/16108/@azure_arm-healthcareapis
branch
2 times, most recently
from
September 21, 2021 23:44
04d913d
to
245f8b3
Compare
openapi-sdkautomation
bot
force-pushed
the
sdkAuto/16108/@azure_arm-healthcareapis
branch
from
October 25, 2021 20:09
245f8b3
to
5caf074
Compare
openapi-sdkautomation
bot
force-pushed
the
sdkAuto/16108/@azure_arm-healthcareapis
branch
8 times, most recently
from
December 8, 2021 23:03
fa2f179
to
ea0d26f
Compare
…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.
…tial service issues (Azure#20105)
* [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
### 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.
Adding owners for automation
* 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
…h violate strict mode and cause a falure as in https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1389880&view=logs&j=dc056dfc-c0cf-5958-c8c4-8da4f91a3739&t=61cf404c-fb38-582c-ffd7-16eb9e7ec7ac&l=97 (Azure#20523) Co-authored-by: Daniel Jurek <djurek@microsoft.com>
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
bot
force-pushed
the
sdkAuto/16108/@azure_arm-healthcareapis
branch
from
February 28, 2022 19:34
ea0d26f
to
7c09960
Compare
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.
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: