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-servicebus] ServiceBus : Swagger Split and Added PrivateEndPointConnection API #1513
Closed
openapi-sdkautomation
wants to merge
127
commits into
sdkAutomation/@azure_arm-servicebus
from
sdkAutomation/@azure_arm-servicebus@10071
Closed
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
4 tasks
b962a92
to
8303e08
Compare
8303e08
to
e557983
Compare
988d41b
to
e95ec74
Compare
Co-authored-by: Zachary Foster <zfoster@users.noreply.github.com>
* Fix missing types in package * Add lint to build and fix lint errors * Let analyze fail with lint errors * Fix lint issue
…entrancy/concurrency issues (Azure#10807) initLink and closeLink had some basic coordination so they wouldn't stomp over each other but they never coordinated via locks which made it so the state of the object would be different than you expected. This commit makes it so both closeLink and openLink depend on the same lock, which allows you to make the modifications to the link state obey ordering. This simplifies some parts of the code that tried to avoid opening and closing simultaneously. As part of this the lock token that was being used in other parts of the code has been subsumed into LinkEntity instead so some modification to the detached/init methods of child classes changed. Also, removed some code in ManagementLink that has become redundant with our unified initLink() code (abort signal handling and some link state checking). Fixes Azure#10656
…les operations (Azure#10884) * Implement async iterators for listEntity * Add list table * Fix extra request * Update API surface * update samples and rollup externals * Add create operation * Update api surface * Clean up * Add details for running tests locally * Make _response non-enumerable * Fix lint errors
* Add foundations for restructuring eng/common directory * Restructure ChangeLog Logic * Reset Common Directory
Azure#10919) * Since those are the only ones supported * Fix types * Add another prop * Update api view Co-authored-by: Ramya Achutha Rao <ramyar@microsoft.com>
* Add tracing * Address feedback
* Separated the tests into internal and public After separating them, I moved some references and started using sinon for mocking instead of having to use some internal function. Tests pass and they should be good for min-max testing, etc. * build fixes
…e#10942) * [Identity] Sovereign Clouds dont support the VSCode credential * feedback by Vinay * throwing also after getTenantIdFromVSCode * Update sdk/identity/identity/src/credentials/visualStudioCodeCredential.ts Co-authored-by: David Wilson <david@daviwil.com> Co-authored-by: David Wilson <david@daviwil.com>
* update ts sample for ai-anomaly-detector * update package.json * remove unsued imports * fix broken link * fix broken links Co-authored-by: yuyi@microsoft.com <Yuanxiang.Ying@microsoft.com>
* javascript example for anomaly detector * javascript samples * update * update * update package * refactor * prettier format Co-authored-by: yuyi@microsoft.com <Yuanxiang.Ying@microsoft.com>
* Added EnableExpress property in CreateQueueOptions and CreateTopicOptions * Update sdk/servicebus/service-bus/src/serializers/queueResourceSerializer.ts Looks good.. not sure how that character is added Co-authored-by: Ramya Rao <ramya.rao.a@outlook.com> * Update sdk/servicebus/service-bus/src/serializers/queueResourceSerializer.ts Co-authored-by: Ramya Rao <ramya.rao.a@outlook.com> * Update sdk/servicebus/service-bus/src/serializers/topicResourceSerializer.ts Co-authored-by: Ramya Rao <ramya.rao.a@outlook.com> * Update sdk/servicebus/service-bus/src/serializers/topicResourceSerializer.ts Co-authored-by: Ramya Rao <ramya.rao.a@outlook.com> * Update sdk/servicebus/service-bus/src/serializers/queueResourceSerializer.ts Co-authored-by: Ramya Rao <ramya.rao.a@outlook.com> * Added enableExpress properties for QueueProperties and TopicProperties interfaces * Update sdk/servicebus/service-bus/src/serializers/topicResourceSerializer.ts Co-authored-by: Ramya Rao <ramya.rao.a@outlook.com> * Update sdk/servicebus/service-bus/src/serializers/queueResourceSerializer.ts Co-authored-by: Ramya Rao <ramya.rao.a@outlook.com> * Updated the service-bus.api documentation * Updated service-bus.api documentation Co-authored-by: Mohsin Mehmood <mohsin@Mohsins-MacBook-Pro.local> Co-authored-by: Ramya Rao <ramya.rao.a@outlook.com> Co-authored-by: Mohsin Mehmood <mohsin85mehmod@gmail.com>
* fix format and check-format commands * format
Convert logging to use the @azure/logger rather than the debug package. This sets us up for properly logging - format changes (and error logging) will come in a future PR. Partial fix for Azure#10638
* [Key Vault Admin] The Backup client source code * Updates based on feedback, specially docs * Apply suggestions from code review Co-authored-by: Christopher Scott <chriscott@hotmail.com> * Param feedback and comments * api review update * small update I did on lro/backup/operation, now on lro/restore/operation * Comment improvement, I think * comment fix * moving the internal properties to an options bag * no need for requestParameters * no need for requestParameters, and some cleanup on the pollers * Addressed Azure#11009 (comment) Co-authored-by: Christopher Scott <chriscott@hotmail.com>
* Increment package version after release of azure-storage-queue * Increment package version after release of azure-storage-file-share * Increment package version after release of azure-storage-blob * Increment package version after release of azure-storage-file-datalake
Co-authored-by: Lin Jian <ljian3377@gmail.com>
* improve datalake and changefeed samples * fix samples to work with dev-tool Co-authored-by: Lin Jian <ljian3377@gmail.com>
12c7732
to
740f027
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.
Created to sync Azure/azure-rest-api-specs#10071.
Installation Instructions
You can install the package
@azure/arm-servicebus
of this PR using the following command:Direct Download
The generated package can be directly downloaded from here: