Skip to content

http-client-java, bug fix duplicate resource name#8176

Merged
weidongxu-microsoft merged 5 commits intomicrosoft:mainfrom
weidongxu-microsoft:http-client-java_bug-fix-duplicate-resource-name
Aug 13, 2025
Merged

http-client-java, bug fix duplicate resource name#8176
weidongxu-microsoft merged 5 commits intomicrosoft:mainfrom
weidongxu-microsoft:http-client-java_bug-fix-duplicate-resource-name

Conversation

@weidongxu-microsoft
Copy link
Copy Markdown
Contributor

see Azure/autorest.java#3182 for detail

@microsoft-github-policy-service microsoft-github-policy-service bot added the emitter:client:java Issue for the Java client emitter: @typespec/http-client-java label Aug 13, 2025
@github-actions
Copy link
Copy Markdown
Contributor

No changes needing a change description found.

@azure-sdk
Copy link
Copy Markdown
Collaborator

azure-sdk commented Aug 13, 2025

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

Copy link
Copy Markdown
Member

@XiaofeiCao XiaofeiCao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit, a comment on the method?

@weidongxu-microsoft weidongxu-microsoft added this pull request to the merge queue Aug 13, 2025
Merged via the queue into microsoft:main with commit b720b08 Aug 13, 2025
26 checks passed
@weidongxu-microsoft weidongxu-microsoft deleted the http-client-java_bug-fix-duplicate-resource-name branch August 13, 2025 10:00
github-merge-queue bot pushed a commit that referenced this pull request Aug 14, 2025
fix problem caused by #8176

It causes problem on `ExtensionResource`, that `systemData` is not
included in the result resource. However it is not reproducible from
azure-http-specs test.

The current solution is to limit this `rename` to `AutoGenerated` class
alone.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emitter:client:java Issue for the Java client emitter: @typespec/http-client-java

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants