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

[AutoPR billing/resource-manager] Swagger for Tenant Properties API (Do #4179

Closed

Conversation

AutorestCI
Copy link
Contributor

added swagger for Tenant Properties

added swagger for Tenant Properties
Fixed camel casing in billing.json file

Fixed camel casing in billing.json file
@codecov-io
Copy link

codecov-io commented Jan 15, 2019

Codecov Report

Merging #4179 into restapi_auto_billing/resource-manager will decrease coverage by <.01%.
The diff coverage is 52.83%.

Impacted file tree graph

@@                            Coverage Diff                            @@
##           restapi_auto_billing/resource-manager    #4179      +/-   ##
=========================================================================
- Coverage                                  56.95%   56.95%   -0.01%     
=========================================================================
  Files                                       8857     8860       +3     
  Lines                                     174838   174891      +53     
=========================================================================
+ Hits                                       99578    99606      +28     
- Misses                                     75260    75285      +25
Impacted Files Coverage Δ
...-billing/azure/mgmt/billing/operations/__init__.py 100% <100%> (ø) ⬆️
...mgmt-billing/azure/mgmt/billing/models/__init__.py 100% <100%> (ø) ⬆️
...ng/azure/mgmt/billing/billing_management_client.py 100% <100%> (ø) ⬆️
.../billing/operations/discover_tenants_operations.py 43.33% <43.33%> (ø)
...lling/azure/mgmt/billing/models/discover_tenant.py 55.55% <55.55%> (ø)
...g/azure/mgmt/billing/models/discover_tenant_py3.py 55.55% <55.55%> (ø)

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 b7fe655...7d2ee3a. Read the comment docs.

@AutorestCI
Copy link
Contributor Author

AutorestCI commented Jan 15, 2019

(message created by the CI based on PR content)

Installation instruction

Package azure-mgmt-consumption

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

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@restapi_auto_5038#egg=azure-mgmt-consumption&subdirectory=azure-mgmt-consumption"

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

  • git checkout restapi_auto_5038
  • pip install -e ./azure-mgmt-consumption

Or build a wheel file to distribute for testing:

  • git checkout restapi_auto_5038
  • pip wheel --no-deps ./azure-mgmt-consumption

Package azure-mgmt-billing

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

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@restapi_auto_5038#egg=azure-mgmt-billing&subdirectory=azure-mgmt-billing"

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

  • git checkout restapi_auto_5038
  • pip install -e ./azure-mgmt-billing

Or build a wheel file to distribute for testing:

  • git checkout restapi_auto_5038
  • pip wheel --no-deps ./azure-mgmt-billing

Direct download

Your files can be directly downloaded here:

AutorestCI and others added 10 commits January 15, 2019 19:48
changes in DiscoverTenants contracts

changes in DiscoverTenants contracts
removal of plural name

removal of plural name
Updating the route

Updating the route as per suggestion
Moved Swagger for Discover Tenants from billing to consumption

Moved Swagger for Discover Tenants from billing to consumption
Change in Namespace followed by one additional parameter "billingAccountId"

Change in Namespace followed by one additional parameter "billingAccountId"
Change of method name from discoverTenants to tanants.

Change of method name from discoverTenants to tanants.
added name after Id. Code review implementation

added name after Id. Code review implementation
@lmazuel
Copy link
Member

lmazuel commented Mar 2, 2019

Will redo PR manually

@lmazuel lmazuel closed this Mar 2, 2019
@lmazuel lmazuel deleted the restapi_auto_5038 branch March 2, 2019 00:57
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.

4 participants