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

Authorization to multi API client #2656

Merged
merged 7 commits into from
Jun 12, 2018
Merged

Authorization to multi API client #2656

merged 7 commits into from
Jun 12, 2018

Conversation

lmazuel
Copy link
Member

@lmazuel lmazuel commented May 30, 2018

No description provided.

@lmazuel lmazuel requested a review from yugangw-msft May 30, 2018 00:09
@AutorestCI
Copy link
Contributor

AutorestCI commented May 30, 2018

(message created by the CI based on PR content)

Installation instruction

Package azure-common

You can install the package azure-common of this PR using the following command:
pip install "git+https://github.com/Azure/azure-sdk-for-python@multi_authorization#egg=azure-common&subdirectory=azure-common"

You can build a wheel to distribute for test using the following command:
pip wheel --no-deps "git+https://github.com/Azure/azure-sdk-for-python@multi_authorization#egg=azure-common&subdirectory=azure-common"

If you have a local clone of this repository, you can also do:

  • git checkout multi_authorization
  • pip install -e ./azure-common

Or build a wheel file to distribute for testing:

  • git checkout multi_authorization
  • pip wheel --no-deps ./azure-common

Package azure-mgmt-authorization

You can install the package azure-mgmt-authorization of this PR using the following command:
pip install "git+https://github.com/Azure/azure-sdk-for-python@multi_authorization#egg=azure-mgmt-authorization&subdirectory=azure-mgmt-authorization"

You can build a wheel to distribute for test using the following command:
pip wheel --no-deps "git+https://github.com/Azure/azure-sdk-for-python@multi_authorization#egg=azure-mgmt-authorization&subdirectory=azure-mgmt-authorization"

If you have a local clone of this repository, you can also do:

  • git checkout multi_authorization
  • pip install -e ./azure-mgmt-authorization

Or build a wheel file to distribute for testing:

  • git checkout multi_authorization
  • pip wheel --no-deps ./azure-mgmt-authorization

Direct download

Your files can be directly downloaded here:

@codecov-io
Copy link

codecov-io commented Jun 7, 2018

Codecov Report

Merging #2656 into master will increase coverage by 0.08%.
The diff coverage is 33.96%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2656      +/-   ##
==========================================
+ Coverage   55.89%   55.97%   +0.08%     
==========================================
  Files        7161     7311     +150     
  Lines      150693   154653    +3960     
==========================================
+ Hits        84227    86571    +2344     
- Misses      66466    68082    +1616
Impacted Files Coverage Δ
...5_07_01/models/provider_operations_metadata_py3.py 36.36% <ø> (ø)
...thorization/v2018_01_01_preview/models/__init__.py 100% <ø> (ø)
..._preview/operations/role_definitions_operations.py 15.04% <ø> (ø)
...01_01_preview/models/role_assignment_filter_py3.py 57.14% <ø> (ø)
...thorization/v2015_07_01/models/permission_paged.py 80% <ø> (ø)
...orization/v2018_01_01_preview/models/permission.py 100% <ø> (ø)
...review/models/role_assignment_create_parameters.py 50% <ø> (ø)
.../v2018_01_01_preview/models/role_assignment_py3.py 38.46% <ø> (ø)
...07_01/models/provider_operations_metadata_paged.py 80% <ø> (ø)
...018_01_01_preview/models/provider_operation_py3.py 36.36% <ø> (ø)
... and 354 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 49020e1...c92ea04. Read the comment docs.

@lmazuel lmazuel merged commit 5444f71 into master Jun 12, 2018
@lmazuel lmazuel deleted the multi_authorization branch June 14, 2018 18:12
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.

3 participants