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.

Sign up for GitHub

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

Jump to bottom

Method signature (alias / spread) Implementation #4543

Closed
Tracked by #612
lmazuel opened this issue Apr 5, 2024 · 2 comments · Fixed by #4893
Closed
Tracked by #612

Method signature (alias / spread) Implementation #4543

lmazuel opened this issue Apr 5, 2024 · 2 comments · Fixed by #4893
Assignees
@archerzz

Comments

@lmazuel
Copy link
Member

lmazuel commented Apr 5, 2024

No description provided.

@lmazuel lmazuel mentioned this issue Apr 5, 2024
Closed
@archerzz
Copy link
Member

This task depends on adoption of TCGC getAllOperations() #3994 . And if we're luck, we don't need to do extra work here.

archerzz pushed a commit to archerzz/autorest.csharp that referenced this issue May 29, 2024
feat(dpg): spread request body model properties as parameters
b6a8303 
- change logic to check `Model.sourceModels` to determine whether the body model should be spread
- regen and update tests.

resolve Azure#4543
@archerzz archerzz mentioned this issue May 29, 2024
Closed
6 tasks
@archerzz
Copy link
Member

archerzz commented Jun 3, 2024

There is a way to not rely on TCGC. Check the PR #4778. The concerns are the breaking changes most of which should be resolved by Azure/typespec-azure#771

@ArcturusZhang ArcturusZhang mentioned this issue Jul 4, 2024
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@archerzz archerzz

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@archerzz @lmazuel