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 sql/resource-manager] Change databaseState in longTermRetention.json to longTermRetentionDatabaseState #2215

Merged
merged 3 commits into from
Mar 19, 2018

Conversation

AutorestCI
Copy link
Contributor

updating databaseState to longTermRetentionDatabaseState
@AutorestCI
Copy link
Contributor Author

AutorestCI commented Mar 19, 2018

(message created by the CI based on PR content)

Installation instruction

Package azure-mgmt-sql

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

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

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

  • git checkout restapi_auto_2693
  • pip install -e ./azure-mgmt-sql

Or build a wheel file to distribute for testing:

  • git checkout restapi_auto_2693
  • pip wheel --no-deps ./azure-mgmt-sql

Direct download

Your files can be directly downloaded here:

Making longTermRetentionDatabaseState a common parameter
@codecov-io
Copy link

codecov-io commented Mar 19, 2018

Codecov Report

Merging #2215 into restapi_auto_sql/resource-manager will increase coverage by 2.23%.
The diff coverage is 26.17%.

Impacted file tree graph

@@                          Coverage Diff                          @@
##           restapi_auto_sql/resource-manager    #2215      +/-   ##
=====================================================================
+ Coverage                              50.76%      53%   +2.23%     
=====================================================================
  Files                                   5581     4783     -798     
  Lines                                 143071   120320   -22751     
=====================================================================
- Hits                                   72633    63777    -8856     
+ Misses                                 70438    56543   -13895
Impacted Files Coverage Δ
...ure-mgmt-sql/azure/mgmt/sql/operations/__init__.py 100% <100%> (ø) ⬆️
...ure/mgmt/sql/models/sql_management_client_enums.py 100% <100%> (ø) ⬆️
azure-mgmt-sql/azure/mgmt/sql/models/__init__.py 100% <100%> (ø) ⬆️
...e-mgmt-sql/azure/mgmt/sql/sql_management_client.py 100% <100%> (ø) ⬆️
...erations/long_term_retention_backups_operations.py 18.11% <18.11%> (ø)
...mt/sql/models/backup_long_term_retention_policy.py 50% <20%> (-5.56%) ⬇️
.../backup_long_term_retention_policies_operations.py 25.28% <20%> (+1.63%) ⬆️
...zure/mgmt/sql/models/long_term_retention_backup.py 41.66% <41.66%> (ø)
...gmt/sql/models/long_term_retention_backup_paged.py 80% <50%> (ø)
...ork/azure/mgmt/network/v2017_10_01/models/probe.py 33.33% <0%> (-8.78%) ⬇️
... and 1011 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 11acc67...10066d0. Read the comment docs.

adding parameter location as method to LongTermRetentionDatabaseStateParameter
@AutorestCI AutorestCI merged commit 7acbb70 into restapi_auto_sql/resource-manager Mar 19, 2018
@AutorestCI
Copy link
Contributor Author

This PR has been merged into #1950

@lmazuel lmazuel deleted the restapi_auto_2693 branch April 27, 2018 01:29
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