Skip to content

[KeyVault] Merging preview branch keyvault_1.0_preview back to master#2817

Merged
schaabs merged 76 commits intomasterfrom
keyvault_1.0_preview
Jun 27, 2018
Merged

[KeyVault] Merging preview branch keyvault_1.0_preview back to master#2817
schaabs merged 76 commits intomasterfrom
keyvault_1.0_preview

Conversation

@schaabs
Copy link

@schaabs schaabs commented Jun 26, 2018

No description provided.

Scott Schaab and others added 30 commits September 22, 2017 10:55
Updating http challenge headers for message protection
[KeyVault] Enabling message protection protocol
[KeyVault] updating autorest generated code from 7.0-preview spec
@schaabs schaabs requested review from lmazuel and tiffanyachen June 26, 2018 21:32
@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@schaabs schaabs force-pushed the keyvault_1.0_preview branch from 1bbb507 to b5d7b6e Compare June 27, 2018 09:46
@codecov-io
Copy link

codecov-io commented Jun 27, 2018

Codecov Report

Merging #2817 into master will increase coverage by 0.38%.
The diff coverage is 82.76%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2817      +/-   ##
==========================================
+ Coverage   56.27%   56.65%   +0.38%     
==========================================
  Files        7472     7598     +126     
  Lines      155707   157788    +2081     
==========================================
+ Hits        87617    89395    +1778     
- Misses      68090    68393     +303
Impacted Files Coverage Δ
...azure/keyvault/models/deleted_secret_item_paged.py 100% <ø> (ø) ⬆️
...keyvault/azure/mgmt/keyvault/models/vault_paged.py 100% <ø> (ø) ⬆️
...lt/azure/keyvault/models/deleted_key_item_paged.py 100% <ø> (ø) ⬆️
...e-keyvault/azure/keyvault/models/key_item_paged.py 100% <ø> (ø) ⬆️
.../keyvault/models/deleted_certificate_item_paged.py 100% <ø> (ø) ⬆️
...eyvault/azure/keyvault/models/secret_item_paged.py 100% <ø> (ø) ⬆️
.../azure/mgmt/keyvault/models/deleted_vault_paged.py 80% <ø> (ø) ⬆️
...e/keyvault/models/certificate_issuer_item_paged.py 100% <ø> (ø) ⬆️
...zure/keyvault/models/storage_account_item_paged.py 80% <ø> (ø) ⬆️
...azure/keyvault/models/sas_definition_item_paged.py 100% <ø> (+20%) ⬆️
... and 367 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 b52e0cf...de49838. Read the comment docs.

@schaabs schaabs force-pushed the keyvault_1.0_preview branch from 4e0cd4c to f2a92ab Compare June 27, 2018 20:45
Copy link
Member

@lmazuel lmazuel left a comment

Choose a reason for hiding this comment

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

Didn't look closely in the details, a few comments on the ChangeLog up to you


Release History
===============
1.0.0b1 (2018-04-10)
Copy link
Member

Choose a reason for hiding this comment

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

Should this file includes a 1.0.0 now? Even if it's just "release 1.0.0b1 as 1.0.0 unchanged"?

1.0.0b1 (2018-04-10)
++++++++++++++++++++

* Upgraded to autorest 3.0 generated code
Copy link
Member

Choose a reason for hiding this comment

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

@schaabs schaabs merged commit 066f421 into master Jun 27, 2018
@lmazuel lmazuel deleted the keyvault_1.0_preview branch August 1, 2018 23:26
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