Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

http-client-java, update test to fill exact param #4770

Conversation

weidongxu-microsoft
Copy link
Contributor

@weidongxu-microsoft weidongxu-microsoft commented Oct 17, 2024

for Azure/cadl-ranch#755, as some tests validate the exact value of some parameter/property.

for the 2 file with only format change, I guess the reason is Alan enabled spotless on emitter Java code (so mvn package would automatically format the code)

@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 Oct 17, 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

@weidongxu-microsoft weidongxu-microsoft marked this pull request as ready for review October 17, 2024 05:15
@weidongxu-microsoft weidongxu-microsoft added this pull request to the merge queue Oct 17, 2024
Merged via the queue into microsoft:main with commit 1b3bdf5 Oct 17, 2024
25 checks passed
@weidongxu-microsoft weidongxu-microsoft deleted the http-client-java_update-test branch October 17, 2024 07:04
swatkatz pushed a commit to swatkatz/typespec that referenced this pull request Nov 5, 2024
for Azure/cadl-ranch#755, as some tests validate
the exact value of some parameter/property.

for the 2 file with only format change, I guess the reason is Alan
enabled spotless on emitter Java code (so `mvn package` would
automatically format the code)
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