Skip to content

http-client-java, fix readonly on flatten property - #8057

Merged
Weidong Xu (weidongxu-microsoft) merged 7 commits into
microsoft:mainfrom
weidongxu-microsoft:http-client-java_fix-flatten-property-readonly
Jul 31, 2025
Merged

http-client-java, fix readonly on flatten property#8057
Weidong Xu (weidongxu-microsoft) merged 7 commits into
microsoft:mainfrom
weidongxu-microsoft:http-client-java_fix-flatten-property-readonly

Conversation

@weidongxu-microsoft

@weidongxu-microsoftWeidong Xu (weidongxu-microsoft) commented Jul 30, 2025

Copy link
Copy Markdown
Contributor

fix#8042

I will need to check regen (and we could still have some case in Swagger SDK) -- this does not commonly happen, but also cannot be ruled out
Azure/autorest.java#3172

@microsoft-github-policy-servicemicrosoft-github-policy-serviceBot added the emitter:client:java Issue for the Java client emitter: @typespec/http-client-java label Jul 30, 2025
@github-actions

Copy link
Copy Markdown
Contributor

No changes needing a change description found.

@azure-sdk

Azure SDK Bot (azure-sdk) commented Jul 30, 2025

Copy link
Copy Markdown
Collaborator

You can try these changes here

🛝 Playground🌐 Website🛝 VSCode Extension

@haolingdong-msft

Copy link
Copy Markdown
Member

Do we want to add a test case?

@weidongxu-microsoft

Weidong Xu (weidongxu-microsoft) commented Jul 31, 2025

Copy link
Copy Markdown
ContributorAuthor

Do we want to add a test case?

Test case added 6f28af6

This fix be a behavior change, but I think the impact should be limited. ARM typically flatten properties property, and this one generally won't be read-only. Xiaofei Cao (@XiaofeiCao)

Merged via the queue into microsoft:main with commit 5849001Jul 31, 2025
26 checks passed
@weidongxu-microsoft
Weidong Xu (weidongxu-microsoft) deleted the http-client-java_fix-flatten-property-readonly branch July 31, 2025 03:38
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[http-client-java] Model readOnly attribute should propagate to its flattened properties

4 participants

@weidongxu-microsoft@azure-sdk@haolingdong-msft@XiaofeiCao