-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Remove explicit version tag from automation #10002
Conversation
@@ -2,7 +2,6 @@ | |||
"$schema": "https://openapistorageprod.blob.core.windows.net/sdkautomation/prod/schemas/swagger_to_sdk_config.schema.json", | |||
"meta": { | |||
"autorest_options": { | |||
"version": "preview", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For now this would need to be 2.0.4413, or 'previous', until the polling for ttags is fixed. Would also need a similar fix in the AzureSDKAutomation fork of the repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version:V2
might be the safest bet.
That at least won't float arbitrarily.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved to "V2" per @fearthecowboy suggestion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Long term, the tags here should be autorest-core tags, but this is the right solution for now.
@markcowl Damn you're right I did the mistake myself..... |
In order to not mess up autorest v3 CLI detection of needed core