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

Add a minimal version of the autorest.nodejs generator for swagger_to… #2598

Merged
1 commit merged into from
Mar 29, 2018

Conversation

ghost
Copy link

@ghost ghost commented Mar 29, 2018

…_sdk to use

@ghost ghost added this to the Sprint-116 milestone Mar 29, 2018
@ghost ghost self-assigned this Mar 29, 2018
@ghost ghost requested review from lmazuel, amarzavery and RikkiGibson March 29, 2018 21:21
@ghost ghost added the in progress label Mar 29, 2018
@@ -3,9 +3,10 @@
"autorest_options": {
"nodejs": "",
"license-header": "MICROSOFT_MIT_NO_VERSION",
"sdkrel:node-sdks-folder": "."
"sdkrel:node-sdks-folder": ".",
"use": "@microsoft.azure/autorest.nodejs@^2.1.43"
Copy link
Member

Choose a reason for hiding this comment

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

Were there issues with old versions of the generator getting used?

Copy link
Author

Choose a reason for hiding this comment

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

Yes, and it was very frustrating.

@ghost ghost merged commit 4a0e750 into master Mar 29, 2018
@ghost ghost deleted the daschult/UpdateMinimalAutorestGeneratorVersion branch March 29, 2018 21:36
@ghost ghost removed the in progress label Mar 29, 2018
This pull request was closed.
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.

1 participant