Skip to content

Conversation

@weidongxu-microsoft
Copy link
Contributor

@weidongxu-microsoft weidongxu-microsoft commented Dec 24, 2024

@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 Dec 24, 2024
@azure-sdk
Copy link
Collaborator

No changes needing a change description found.

@azure-sdk
Copy link
Collaborator

You can try these changes here

🛝 Playground 🌐 Website 📚 Next docs 🛝 VSCode Extension

@weidongxu-microsoft weidongxu-microsoft changed the title http-client-java, generate mock long for BigDecimal http-client-java, generate mock Long number for BigDecimal Dec 24, 2024
@weidongxu-microsoft weidongxu-microsoft marked this pull request as ready for review December 24, 2024 07:42
@weidongxu-microsoft
Copy link
Contributor Author

weidongxu-microsoft commented Jan 3, 2025

@XiaofeiCao I've changed the impl to just skip the test case, when "constant string too long" could happen (as it likely also trigger "code too large").

@weidongxu-microsoft weidongxu-microsoft added this pull request to the merge queue Jan 3, 2025
Merged via the queue into microsoft:main with commit 8f37063 Jan 3, 2025
26 checks passed
@weidongxu-microsoft weidongxu-microsoft deleted the http-client-java_fix-mock-test branch January 3, 2025 08:49
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