Skip to content
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

[TCGC] Initial Adoption of getAllModels() #4122

Closed
archerzz opened this issue Jan 11, 2024 · 1 comment
Closed

[TCGC] Initial Adoption of getAllModels() #4122

archerzz opened this issue Jan 11, 2024 · 1 comment
Assignees
Labels
DPG/RLC v2.1 Post Gallium work DPG v3 Version 3 of AutoRest C# generator.

Comments

@archerzz
Copy link
Member

Description

As a pre-requisite of #3994, we should first adopt getAllModels().

Describe your ideas for solutions
Leverage closed PR (#3627). Ideally, we should be able to just convert return value of getAllModels() into input types. But the model hierarchy of TCGC could be slightly different from model hierarchy generated by csharp emitter. So, we'd better first just use getAllModels() as a source of models when iterating operations/models in the emitter.

@archerzz archerzz added v3 Version 3 of AutoRest C# generator. DPG labels Jan 11, 2024
@archerzz archerzz self-assigned this Jan 11, 2024
@archerzz archerzz added the DPG/RLC v2.1 Post Gallium work label Jan 22, 2024
@archerzz
Copy link
Member Author

close since it's a duplicate of #4300

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DPG/RLC v2.1 Post Gallium work DPG v3 Version 3 of AutoRest C# generator.
Projects
None yet
Development

No branches or pull requests

1 participant