8000 [AzureRG Deploy] Removing multiple occurrences of task-lib by thesattiraju · Pull Request #4619 · microsoft/azure-pipelines-tasks · GitHub
[go: up one dir, main page]

Skip to content

[AzureRG Deploy] Removing multiple occurrences of task-lib #4619

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

Merged
merged 3 commits into from
Jun 23, 2017

Conversation

thesattiraju
Copy link
Contributor

No description provided.

"rm": [
{
"items": [
"node_modules/azure-arm-rest/node_modules/vsts-task-lib"
Copy link
Member

Choose a reason for hiding this comment

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

Don't we need the same to vso-node-api ?

Copy link
Contributor

Choose a reason for hiding this comment

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

vso-node-api does not do any encryption.

},
{
"items": [
"node_modules/azure-arm-rest/node_modules/vso-node-api"
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't you merge with the above array. Items say it can expect an array.

@thesattiraju thesattiraju merged commit 92dfffd into releases/m119 Jun 23, 2017
@thesattiraju thesattiraju deleted the users/desattir/encryptionFix branch June 23, 2017 08:05
bishal-pdMSFT added a commit that referenced this pull request Jun 23, 2017
…mmit #92dfffdcecaef57e782ad666b47b1faabd940937
bishal-pdMSFT added a commit that referenced this pull request Jun 23, 2017
…kHotfixMaster

[AzureRG Deploy] Removing multiple occurrences of task-lib (#4619) Co…
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.

4 participants
0