-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[EventHubs] improve code coverage #19553
Conversation
/azp run python - eventhub - tests |
No pipelines are associated with this pull request. |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run python - eventhub - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run python - eventhub - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run python - eventhub - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run python - eventhub - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
…om/swathipil/azure-sdk-for-python into swathipil/eh/improve-code-coverage
/azp run python - eventhub - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run python - eventhub - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
sdk/eventhub/azure-eventhub/azure/eventhub/_connection_manager.py
Outdated
Show resolved
Hide resolved
sdk/eventhub/azure-eventhub/azure/eventhub/aio/_connection_manager_async.py
Outdated
Show resolved
Hide resolved
/azp run python - eventhub - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run python - eventhub - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run python - eventhub - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
Hello @swathipil! Because this pull request has the 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 (
|
/azp run python - eventhub - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
Added first stable version for managed resolver (Azure#19553) * Added stable version for managed resolver * Same changes as preview swagger * Revert "Same changes as preview swagger" This reverts commit 7ce9c574385df538883e418b6e0c98a4bd114e83. * Fix prettier checks * Renamed the API version to 2022-07-01 Co-authored-by: Arpit Gagneja <[email protected]>
part of #19238