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

Node6hotfixes/azure file copy #14147

Merged
merged 17 commits into from
Feb 24, 2021

Conversation

agrataru
Copy link
Contributor

Task name: AzureFileCopy V2, V3 and V4

Description:

  • Updating the node version to node 10
  • Bumping up the minor version

Testing

Windows Agent

AzureFileCopyV2

  • Tested the blob upload and VM upload

AzureFileCopyV3

  • Tested the blob upload and VM upload

AzureFileCopyV4

  • Tested the blob upload and VM upload

Documentation changes required: (Y/N)

Added unit tests: (Y/N)

Attached related issue: (Y/N)

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

@@ -1,12 +1,15 @@
{
"name": "AzureFileCopyV2",
"dependencies": {
"@types/mocha": "2.2.48",
"@types/node": "6.0.68",
"@types/mocha": "^5.2.7",
Copy link
Member

Choose a reason for hiding this comment

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

You should stick to a specific version. for all the packages.

"@types/q": "1.0.7",
"azure-pipelines-tasks-azure-arm-rest-v2": "1.0.4",
"azure-pipelines-task-lib": "2.8.0",
"azure-pipelines-task-lib": "3.0.6-preview.0",
Copy link
Member

Choose a reason for hiding this comment

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

is there a stable version of this library that we can use ? Or do we have to use this version ?

Copy link
Member

Choose a reason for hiding this comment

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

@arjgupta Hi! This is the version prepared for Node 10 migration.

@@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 2,
"Minor": 178,
"Minor": 179,
Copy link
Member

Choose a reason for hiding this comment

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

Could you please bump the version in accordance with the current sprint as per this guide for all tasks changed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@agrataru agrataru merged commit 61e4114 into microsoft:master Feb 24, 2021
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.

3 participants