Skip to content

Conversation

@sarangan12
Copy link
Contributor

Based on request from @lmazuel (Issue: https://github.com/Azure/sdk-release-request/issues/797) , I have regenerated the Cognitive Services Computer Vision TS SDK in this PR. Command Used:

autorest --reset && autorest --typescript --license-header=MICROSOFT_MIT_NO_VERSION --typescript-sdks-folder=C:\Users\sarajama\Projects\azure-sdk-for-js [email protected]/[email protected] --package-version=7.1.0 https://raw.githubusercontent.com/Azure/azure-rest-api-specs/master/specification/cognitiveservices/data-plane/ComputerVision/readme.md

As you can see from the PR, only the service version info changed. Other than that, the only other information is 2 new comments. Updated the version 7.0.1 -> 7.1.0.

@lmazuel FYI
@ramya-rao-a Please review and approve.

Copy link
Member

@joheredi joheredi left a comment

Choose a reason for hiding this comment

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

It looks good, I'm curious what caused the formatting changes. It looks like the changes in this PR honor the formatting that the generator outputs. We may want to ignore generated code in the tooling that re-formatted the code prior to this PR

@sarangan12
Copy link
Contributor Author

Reverted the formatting changes. Keeping only the generated code

@sarangan12 sarangan12 merged commit 688c412 into Azure:master Oct 9, 2020
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-js that referenced this pull request Nov 19, 2020
Reverting the changed made for the address space update in peered vNets. PR: 11090 (Azure#11731)

Co-authored-by: Hari Prasad Perabattula <[email protected]>
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.

2 participants