-
Notifications
You must be signed in to change notification settings - Fork 160
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
Emitter needs to adopt TCGC's recent changes on nullable
#4701
Labels
Comments
ArcturusZhang
changed the title
tcgc label nullnable to property/parameter instead of type, Codegen need to adapt
tcgc label nullable to property/parameter instead of type, Codegen need to adapt
May 13, 2024
ArcturusZhang
changed the title
tcgc label nullable to property/parameter instead of type, Codegen need to adapt
tcgc label nullable to property/parameter instead of type, Codegen need to adopt
May 13, 2024
ArcturusZhang
changed the title
tcgc label nullable to property/parameter instead of type, Codegen need to adopt
Emitter needs to adopt TCGC's recent changes on May 13, 2024
nullable
we could take this opportunity to review and refine the design of our input types as well |
Also this should fix #4314 |
Latest decision is to deprecate all |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
TCGC will label
Nullable
on property/parameter instead of type.Codegen emitter will adapt this:
Nullable
on property/parameterNullable
in CsharpType, update the TypeFactory.cs to passNullable
parameter when create CsharpType.The text was updated successfully, but these errors were encountered: