Skip to content

Comments

[feature] added print stataement#7

Open
pushpitkamboj wants to merge 1 commit intomasterfrom
feature/print_statement
Open

[feature] added print stataement#7
pushpitkamboj wants to merge 1 commit intomasterfrom
feature/print_statement

Conversation

@pushpitkamboj
Copy link
Collaborator

added a print statement and which will tell me how is it going.
Fixed #2

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

Adds a user-visible message intended to indicate the program’s purpose/progress for the pushpit script (related to the basic demo program introduced in prior work).

Changes:

  • Added a new print() message describing the program.

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

Comment on lines +34 to 36
print("This is a simple Python program demonstrating basic concepts.")

if __name__ == "__main__":
Copy link

Copilot AI Feb 16, 2026

Choose a reason for hiding this comment

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

This top-level print introduces a module import side effect and will run even when pushpit.main is imported (e.g., from tests/other modules). If the intent is to show this message when executing the script, move it into main() or inside the if __name__ == "__main__": block (and avoid duplicating the module docstring).

Suggested change
print("This is a simple Python program demonstrating basic concepts.")
if __name__ == "__main__":
if __name__ == "__main__":
print("This is a simple Python program demonstrating basic concepts.")

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.

1 participant