Skip to content

Conversation

@seongm1n
Copy link
Contributor

@seongm1n seongm1n commented Aug 10, 2025

Removes unnecessary System.out.println(systemMessage.getText()) from test method.

The debug output is redundant since the same value is validated by the assertion on the next line. This improves code quality and follows Spring project conventions of using proper logging instead of console output.

File changed:

  • QuestionAnswerAdvisorTests.java (line 154)

Testing: Test continues to pass with no functional impact.

Auto-cherry-pick to 1.0.x
Fixes #4102

spring-builds pushed a commit that referenced this pull request Aug 11, 2025
Signed-off-by: Theo <[email protected]>
Signed-off-by: seongm1n <[email protected]>

Fixes #4102

(cherry picked from commit c72c8c2)
@markpollack markpollack self-assigned this Aug 11, 2025
tzolov pushed a commit to tzolov/spring-ai that referenced this pull request Aug 17, 2025
scionaltera pushed a commit to scionaltera/spring-ai that referenced this pull request Sep 3, 2025
chedim pushed a commit to couchbaselabs/spring-ai that referenced this pull request Sep 19, 2025
Willam2004 pushed a commit to Willam2004/spring-ai that referenced this pull request Oct 11, 2025
Signed-off-by: Theo <[email protected]>
Signed-off-by: seongm1n <[email protected]>

Auto-cherry-pick to 1.0.x
Fixes spring-projects#4102

Signed-off-by: 家娃 <[email protected]>
ralla0405 pushed a commit to ralla0405/spring-ai that referenced this pull request Nov 28, 2025
Signed-off-by: Theo <[email protected]>
Signed-off-by: seongm1n <[email protected]>

Auto-cherry-pick to 1.0.x
Fixes spring-projects#4102

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants