-
Notifications
You must be signed in to change notification settings - Fork 40
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
Enums are always fixed after we switch to use union to represent extensible enum #459
Conversation
All changed packages have been documented. |
You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/459/ Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/459/ |
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.
Before we merge that I think we need to also have everything else ready to merge(Not need to be part of your PR),
- linter rule already enabled
- autorest also changing behavior
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.
It was decided today during the meeting that we'll get that in for this sprint
Made the autorest change #473
Co-authored-by: Timothee Guerin <timothee.guerin@outlook.com>
Change TCGC behavior.