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 dns/resource-manager] Use common arm definitions for dns #3004

Merged
merged 1 commit into from
Jul 25, 2018

Conversation

AutorestCI
Copy link
Contributor

@AutorestCI
Copy link
Contributor Author

AutorestCI commented Jul 25, 2018

(message created by the CI based on PR content)

Installation instruction

Package azure-mgmt-dns

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

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_3502#egg=azure-mgmt-dns&subdirectory=azure-mgmt-dns"

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

  • git checkout restapi_auto_3502
  • pip install -e ./azure-mgmt-dns

Or build a wheel file to distribute for testing:

  • git checkout restapi_auto_3502
  • pip wheel --no-deps ./azure-mgmt-dns

Direct download

Your files can be directly downloaded here:

@codecov-io
Copy link

codecov-io commented Jul 25, 2018

Codecov Report

Merging #3004 into restapi_auto_dns/resource-manager will increase coverage by <.01%.
The diff coverage is 88.05%.

Impacted file tree graph

@@                          Coverage Diff                          @@
##           restapi_auto_dns/resource-manager    #3004      +/-   ##
=====================================================================
+ Coverage                              56.35%   56.36%   +<.01%     
=====================================================================
  Files                                   8124     8130       +6     
  Lines                                 166282   166326      +44     
=====================================================================
+ Hits                                   93714    93752      +38     
- Misses                                 72568    72574       +6
Impacted Files Coverage Δ
...re/mgmt/dns/v2018_03_01_preview/models/resource.py 100% <ø> (ø) ⬆️
...t/dns/v2018_03_01_preview/dns_management_client.py 36.66% <ø> (ø) ⬆️
...re/mgmt/dns/v2018_03_01_preview/models/zone_py3.py 100% <100%> (ø) ⬆️
...dns/v2018_03_01_preview/models/tracked_resource.py 100% <100%> (ø)
...re/mgmt/dns/v2018_03_01_preview/models/__init__.py 100% <100%> (ø) ⬆️
...v2018_03_01_preview/models/tracked_resource_py3.py 100% <100%> (ø)
.../azure/mgmt/dns/v2018_03_01_preview/models/zone.py 100% <100%> (ø) ⬆️
...gmt/dns/v2018_03_01_preview/models/resource_py3.py 100% <100%> (ø) ⬆️
...2018_03_01_preview/models/azure_entity_resource.py 71.42% <71.42%> (ø)
..._03_01_preview/models/azure_entity_resource_py3.py 71.42% <71.42%> (ø)
... and 10 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 275c84d...3dd9ba3. Read the comment docs.

@AutorestCI AutorestCI merged commit d1672f1 into restapi_auto_dns/resource-manager Jul 25, 2018
@AutorestCI
Copy link
Contributor Author

This PR has been merged into #3000

AutorestCI added a commit that referenced this pull request Aug 1, 2018
lmazuel pushed a commit that referenced this pull request Aug 1, 2018
* Generated from c5f0cf3de92783c4d87720bc73686bd19fbb79d3 (#2998)

Added CreatePrivateZone example ref to swagger spec

* Generated from 4de933ca3a2caaa1c978a697a768c9bf5bdf35b8 (#3004)

Extract go config

* DNS 2.0.0 packaging
@lmazuel lmazuel deleted the restapi_auto_3502 branch August 1, 2018 23:22
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.

2 participants