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

[ServiceBus] remove msrest #29324

Merged
19 commits merged into from
Mar 24, 2023
Merged

[ServiceBus] remove msrest #29324

19 commits merged into from
Mar 24, 2023

Conversation

swathipil
Copy link
Member

@swathipil swathipil commented Mar 13, 2023

@ghost ghost added the Service Bus label Mar 13, 2023
@swathipil
Copy link
Member Author

/azp run python - servicebus - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@azure-sdk
Copy link
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

azure-servicebus

@swathipil
Copy link
Member Author

/azp run python - servicebus - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@swathipil swathipil marked this pull request as ready for review March 21, 2023 16:03
Copy link
Member

@l0lawrence l0lawrence left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@swathipil swathipil marked this pull request as draft March 22, 2023 16:17
@swathipil
Copy link
Member Author

/azp run python - servicebus - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@lmazuel lmazuel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quick review LGTM!

@swathipil
Copy link
Member Author

/azp run python - servicebus - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@swathipil swathipil marked this pull request as ready for review March 24, 2023 16:53
@ghost
Copy link

ghost commented Mar 24, 2023

Hello @swathipil!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit f9c4c24 into Azure:main Mar 24, 2023
@swathipil swathipil deleted the swathipil/sb/remove-msrest branch March 24, 2023 21:48
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request Mar 27, 2023
…into network_ga

* 'main' of https://github.com/Azure/azure-sdk-for-python: (63 commits)
  check only once whether artifact extension exists (Azure#29594)
  code and test (Azure#29530)
  fix: support data binding expression for resources.xxx (Azure#29559)
  code and test (Azure#29558)
  [ServiceBus] enable sovereign cloud testing (Azure#29202)
  [AutoRelease] t2-communication-2023-03-20-22792(can only be merged by SDK owner) (Azure#29453)
  [Identity] Allow configurable process timeouts (Azure#28290)
  Increment package version after release of azure-containerregistry (Azure#29582)
  Update optional (Azure#28435)
  [Monitor][Query] Add sample notebook for large queries (Azure#28148)
  Update CHANGELOG.md (Azure#29580)
  [ServiceBus] remove msrest (Azure#29324)
  Explicitly set PSNativeCommandArgumentPassing to Legacy for git push script (Azure#29554)
  [ML][Pipelines] Support queue settings in pipeline job (Azure#29209)
  add experimental decorator for datatransfer (Azure#29562)
  Featurestore/online store crud new pr (Azure#29546)
  switch tooling over to typespec (Azure#29501)
  set up base path for feature set in load function (Azure#29544)
  Address credential description in docstring (Azure#29549)
  [Monitor] Regen query and ingestion (Azure#29504)
  ...
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants