Skip to content

Conversation

@yuluo-yx
Copy link
Contributor

optimize: use already defined variables

@dev-jonghoonpark
Copy link
Contributor

@yuluo-yx

The log shows the following

Run `spring-javaformat:apply` to fix.

@yuluo-yx
Copy link
Contributor Author

I run mvn spring-javaformat:apply, but no any changes for my local code. 🥲

image

@dev-jonghoonpark
Copy link
Contributor

dev-jonghoonpark commented Mar 17, 2025

The github action logs show that there is a problem with OpenAiChatModel.java

Error:  Failed to execute goal io.spring.javaformat:spring-javaformat-maven-plugin:0.0.43:validate (default) on project spring-ai-openai: Formatting violations found in the following files:
Error:   * /home/runner/work/spring-ai/spring-ai/models/spring-ai-openai/src/main/java/org/springframework/ai/openai/OpenAiChatModel.java

and when I test it locally, the change occurs at line 199

image

(But I think the left side is cleaner.)

@yuluo-yx
Copy link
Contributor Author

yep, me too. emm~ I will try to fix.

@ilayaperumalg
Copy link
Member

@yuluo-yx Thanks for the code cleanup!

@ilayaperumalg ilayaperumalg self-assigned this Mar 17, 2025
@ilayaperumalg ilayaperumalg added this to the 1.0.0-M7 milestone Mar 17, 2025
@ilayaperumalg ilayaperumalg merged commit 0164b5d into spring-projects:main Mar 17, 2025
2 checks passed
@yuluo-yx yuluo-yx deleted the 0315-yuluo/optimize-1 branch March 18, 2025 05:09
leijendary pushed a commit to leijendary/spring-ai that referenced this pull request Mar 28, 2025
* optimize: use already defined variables

Signed-off-by: yuluo-yx <[email protected]>
chedim pushed a commit to couchbaselabs/spring-ai that referenced this pull request Sep 19, 2025
* optimize: use already defined variables

Signed-off-by: yuluo-yx <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants