Skip to content

[http-client-java] mgmt, remove @Generated annotation - #7268

Closed
Xiaofei Cao (XiaofeiCao) wants to merge 10 commits into
mainfrom
mgmt_remove_generated_annotation
Closed

Xiaofei Cao (XiaofeiCao) wants to merge 10 commits into
mainfrom
mgmt_remove_generated_annotation

Conversation

@XiaofeiCao

@XiaofeiCao Xiaofei Cao (XiaofeiCao) commented May 7, 2025 •

Copy link
Copy Markdown
Member

Fixed a bug hidden behind isDataPlane check in addGeneratedAnnotation, that's now removed in VNext PR.

FluentModelTemplate is used when sss is disabled. FluentStreamStyleSerializationModelTemplate is the one when sss is enabled.

@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 May 7, 2025
@github-actions

github-actions Bot commented May 7, 2025

Copy link
Copy Markdown
Contributor

No changes needing a change description found.

@azure-sdk

Azure SDK Bot (azure-sdk) commented May 7, 2025 •

Copy link
Copy Markdown
Collaborator

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

@XiaofeiCao
Xiaofei Cao (XiaofeiCao) removed this pull request from the merge queue due to a manual request May 7, 2025
github-merge-queue Bot pushed a commit that referenced this pull request May 7, 2025
`isAzure` is very confusing, as we now have `isAzureV1`, `isAzureV2`
(that I thought this `isAzure` means "either v1 or v2", but it is not).
It should not have effect in track2. We always use `--fluent` hence
`isFluent` or `isAzureOrFluent` (this get removed too).

merged with #7268

add adaption to tcgc 0.56.1, as it affects behavior

---------

Co-authored-by: XiaofeiCao <xiaofeicao@microsoft.com>
@XiaofeiCao

Copy link
Copy Markdown
Member Author

Closed by #7267

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