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

New ServiceBus SDK #4165

Merged
merged 4 commits into from
Jan 18, 2019
Merged

New ServiceBus SDK #4165

merged 4 commits into from
Jan 18, 2019

Conversation

annatisch
Copy link
Member

No description provided.

@annatisch annatisch requested a review from lmazuel January 11, 2019 22:50
@adxsdk6
Copy link

adxsdk6 commented Jan 11, 2019

Can one of the admins verify this patch?

@codecov-io
Copy link

codecov-io commented Jan 15, 2019

Codecov Report

Merging #4165 into master will decrease coverage by 0.26%.
The diff coverage is 32.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4165      +/-   ##
==========================================
- Coverage   54.04%   53.77%   -0.27%     
==========================================
  Files        9838     9853      +15     
  Lines      207611   209786    +2175     
==========================================
+ Hits       112194   112815     +621     
- Misses      95417    96971    +1554
Impacted Files Coverage Δ
.../azure/servicebus/control_client/_http/__init__.py 100% <ø> (ø)
azure-servicebus/azure/__init__.py 100% <ø> (ø)
...cebus/azure/servicebus/control_client/constants.py 100% <100%> (ø)
...icebus/azure/servicebus/control_client/__init__.py 100% <100%> (ø)
.../azure/servicebus/control_client/_serialization.py 98.76% <100%> (ø)
azure-servicebus/azure/servicebus/__init__.py 100% <100%> (ø) ⬆️
...re-servicebus/azure/servicebus/common/constants.py 100% <100%> (ø)
...rvicebus/azure/servicebus/control_client/models.py 94.8% <100%> (ø)
azure-servicebus/azure/servicebus/aio/__init__.py 100% <100%> (ø)
...ervicebus/azure/servicebus/common/mgmt_handlers.py 15.51% <15.51%> (ø)
... and 48 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3390380...ba407e4. Read the comment docs.

import functools

from azure.servicebus.common import message
from azure.servicebus.common.utils import get_running_loop
Copy link
Member

Choose a reason for hiding this comment

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

Note for myself and FYI to you: I shall create a azure.common.async_tools with it instead of keeping this SB only

@@ -0,0 +1,45 @@
[MASTER]
Copy link
Member

Choose a reason for hiding this comment

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

Is this the CLI one, or a SB specific one?

Copy link
Member Author

Choose a reason for hiding this comment

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

It is the CLI one with one extra suppression: useless-object-inheritance

azure-servicebus/setup.py Outdated Show resolved Hide resolved
Copy link
Contributor

@mmacy mmacy left a comment

Choose a reason for hiding this comment

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

A few changes requested, Anna. Many (most?) of them are easy find+replace ops.

azure-servicebus/HISTORY.rst Outdated Show resolved Hide resolved
@@ -1,222 +1,85 @@
Microsoft Azure SDK for Python
==============================

This is the Microsoft Azure Service Bus Runtime Client Library.
This is the Microsoft Azure ServiceBus Client Library.
Copy link
Contributor

Choose a reason for hiding this comment

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

Recommend converting this README to Markdown, and reusing the content from the Service Bus landing page you've got in the other PR. Using the same content for the README, SDK landing page on docs, and the PyPi distribution page is the guidance we'll be putting forth, and is what we've standardized on based on the findings from the Python SDK UX studies.

See guidance here: Document your Azure SDK.

Copy link
Member Author

Choose a reason for hiding this comment

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

I've updated the readme to be closer to the docs.ms.com overview - keeping as rst for first release as discussed offline, with subsequent move to markdown.

azure-servicebus/azure/servicebus/aio/async_client.py Outdated Show resolved Hide resolved
azure-servicebus/azure/servicebus/aio/async_client.py Outdated Show resolved Hide resolved
azure-servicebus/azure/servicebus/aio/async_client.py Outdated Show resolved Hide resolved
azure-servicebus/azure/servicebus/send_handler.py Outdated Show resolved Hide resolved
azure-servicebus/azure/servicebus/send_handler.py Outdated Show resolved Hide resolved
azure-servicebus/azure/servicebus/servicebus_client.py Outdated Show resolved Hide resolved
azure-servicebus/azure/servicebus/servicebus_client.py Outdated Show resolved Hide resolved
azure-servicebus/azure/servicebus/aio/async_client.py Outdated Show resolved Hide resolved
@annatisch
Copy link
Member Author

Hi @lmazuel and @mmacy - I think I have addressed all of your feedback.
Please take another look and sign off when ready :)

Copy link
Contributor

@mmacy mmacy left a comment

Choose a reason for hiding this comment

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

Approving with two small change requests. 😊

azure-servicebus/README.rst Show resolved Hide resolved
azure-servicebus/README.rst Show resolved Hide resolved
@annatisch annatisch changed the title [DoNotMerge] New ServiceBus SDK New ServiceBus SDK Jan 17, 2019
* Docs updates

* Some docs cleanup

* More docs fixes

* Fixed docs indentation

* More docs updates

* Added Example tags

* More docstrings

* Fixed return type

* Update CODEOWNERS

* Test fixes
* Review feedback

* Moved pylint config

* Test stabilization

* Updated readme

* SB capitalization

* Docstring fixes

* Block comments

* Flake8 docs pass

* Pylint fix
@lmazuel lmazuel merged commit 7105cb6 into master Jan 18, 2019
@lmazuel lmazuel deleted the servicebus branch January 18, 2019 00:32
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.

5 participants