Skip to content
This repository was archived by the owner on May 5, 2023. It is now read-only.

Update AKV generated code#2456

Merged
amarzavery merged 2 commits intoAzure:keyvaultfrom
daviddesberg:feature/autorest_update
Mar 13, 2018
Merged

Update AKV generated code#2456
amarzavery merged 2 commits intoAzure:keyvaultfrom
daviddesberg:feature/autorest_update

Conversation

@daviddesberg
Copy link
Contributor

  • Management plane: to latest 2016-10-01
  • Data plane: to 7.0-preview

@amarzavery
Copy link
Contributor

@lusitanian - Can you please update the semantic version of the package in lib/services/keyVault/package.json. Also make sure to update the versions of "ms-rest" and "ms-rest-azure" to the latest version respectively.

Copy link

@schaabs schaabs left a comment

Choose a reason for hiding this comment

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

:shipit:

@amarzavery amarzavery merged commit 3dd5295 into Azure:keyvault Mar 13, 2018
@amarzavery
Copy link
Contributor

@lusitanian - Please publish the "azure-arm-keyvault" and "azure-keyvault" packages to npm, following the instructions over here.

@daviddesberg daviddesberg deleted the feature/autorest_update branch March 13, 2018 17:46
@daviddesberg
Copy link
Contributor Author

@amarzavery the jenkins build seems to fail when i try publishing either package because it's unable to touch this directory:

java.io.IOException: Failed to mkdirs: D:/workspace/sdk-for-node-publish

@amarzavery
Copy link
Contributor

interesting. @shawns1 - Can you please take a look a this issue?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants