Skip to content

Comments

Vihobranch#3

Open
HUANGV1 wants to merge 2 commits intossm-lab:mainfrom
aksh-13:vihobranch
Open

Vihobranch#3
HUANGV1 wants to merge 2 commits intossm-lab:mainfrom
aksh-13:vihobranch

Conversation

@HUANGV1
Copy link

@HUANGV1 HUANGV1 commented Jan 6, 2026

yes yes yes

Copilot AI review requested due to automatic review settings January 6, 2026 18:48
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR modifies the output messages in two HelloWorld implementations, changing from the standard "Hello, World!" greeting to include additional exclamatory text.

  • Updates Python HelloWorld to append "KING QUEEN!" to the output
  • Updates Java HelloWorld to append "Hello HELLO!" to the output

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
HelloWorld.py Modified print statement to add "KING QUEEN!" after "Hello, World!"
HelloWorld.java Modified System.out.print statement to add "Hello HELLO!" after "Hello, World!"

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

public class HelloWorld{
public static void main(String[] args) {
System.out.print("Hello, World!");
System.out.print("Hello, World, Hello HELLO!");
Copy link

Copilot AI Jan 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comma placement creates a grammatical issue. There should not be a comma before "Hello HELLO!" as it creates an unnatural pause. Consider either removing the comma or rephrasing the message for clarity.

Suggested change
System.out.print("Hello, World, Hello HELLO!");
System.out.print("Hello, World Hello HELLO!");

Copilot uses AI. Check for mistakes.
@@ -1 +1 @@
print("Hello, World!") No newline at end of file
print("Hello, World, KING QUEEN!") No newline at end of file
Copy link

Copilot AI Jan 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The output messages are inconsistent between the Python and Java implementations. One prints "KING QUEEN!" while the other prints "Hello HELLO!". Consider using the same message across both files to maintain consistency in the application's output.

Suggested change
print("Hello, World, KING QUEEN!")
print("Hello HELLO!")

Copilot uses AI. Check for mistakes.
@@ -1 +1 @@
print("Hello, World!") No newline at end of file
print("Hello, World, KING QUEEN!") No newline at end of file
Copy link

Copilot AI Jan 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comma placement creates a grammatical issue. There should not be a comma before "KING QUEEN" as it creates an unnatural pause. Consider either removing the comma or rephrasing the message for clarity.

Suggested change
print("Hello, World, KING QUEEN!")
print("Hello, World KING QUEEN!")

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants