Skip to content

Fix LRO operation status model not generation issue#3434

Merged
MaryGao merged 11 commits intoAzure:mainfrom
v-jiaodi:test812
Oct 10, 2025
Merged

Fix LRO operation status model not generation issue#3434
MaryGao merged 11 commits intoAzure:mainfrom
v-jiaodi:test812

Conversation

@v-jiaodi
Copy link
Copy Markdown
Member

@v-jiaodi v-jiaodi commented Aug 12, 2025 •
edited by MaryGao
Loading

fixes #3433

We don't need to azure core lro model filter out logic now, because

  • if it is an LRO, the tcgc would prepare the correct return type for us
  • if it is not an LRO, the direct referenced azure core model e.g operationstate shoud be generated

@MaryGao MaryGao changed the title Verify that the library option is set to true for @azure-rest/azure-widget-analytics Issue found during DPG Modular generation - placeholder generated Sep 10, 2025
@MaryGao MaryGao changed the title Issue found during DPG Modular generation - placeholder generated Fix LRO operation status model not generation issue Sep 29, 2025
@MaryGao MaryGao marked this pull request as ready for review September 29, 2025 08:43
@MaryGao MaryGao enabled auto-merge (squash) October 10, 2025 06:53
@MaryGao MaryGao merged commit d650bb5 into Azure:main Oct 10, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MaryGao MaryGao MaryGao approved these changes

@joheredi joheredi Awaiting requested review from joheredi joheredi is a code owner

@qiaozha qiaozha Awaiting requested review from qiaozha qiaozha is a code owner

Assignees

@MaryGao MaryGao

@v-jiaodi v-jiaodi

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Issue found during DPG modular generation - placeholder generated

2 participants

@v-jiaodi @MaryGao
⚡