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

Conversation

@xingwu1
Copy link
Member

@xingwu1 xingwu1 commented Mar 10, 2018

No description provided.

@xingwu1
Copy link
Member Author

xingwu1 commented Mar 10, 2018

The swagger spec in in PR: Azure/azure-rest-api-specs#2556

@xingwu1
Copy link
Member Author

xingwu1 commented Mar 10, 2018

@amarzavery Could you review my PR or add other reviewer?

"ms-rest-azure": "^2.3.3",
"underscore": "^1.8.3",
"moment": "~2.18.1",
"moment": "^2.18.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

Please change this to "^2.21.0". This will make sure we will have a version higher than 2.18.1 (2.1.8.1 has security vulnerabilities ).

Copy link
Contributor

Choose a reason for hiding this comment

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

@xingwu1 - Also update the minimum versions of ms-rest and ms-rest-azure to their latest versions ^2.3.2 and ^2.5.4 respectively.

"ms-rest-azure": "^2.5.4",
"underscore": "^1.8.3",
"moment": "^2.18.1",
"moment": "^2.21.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

@xingwu1 - this should be "^2.21.0". As per npm that is the latest version of npm that has been published. 2.21.1 is not yet available.

@amarzavery
Copy link
Contributor

@xingwu1 - The PR of ms-rest has been merged and the package has been published.

@amarzavery
Copy link
Contributor

@xingwu1 - Can this PR be merged?

@xingwu1 xingwu1 changed the title [Do not merge] Update Batch SDK to API version 2018-03-01.6.1 Update Batch SDK to API version 2018-03-01.6.1 Mar 19, 2018
@xingwu1
Copy link
Member Author

xingwu1 commented Mar 19, 2018

@amarzavery We are ready to be merged. The official swagger spec has been merged.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants