Skip to content

TEST PR, Do not Merge!!! #20

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.

Sign up for GitHub

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

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
+11 −0
Open

TEST PR, Do not Merge!!! #20

Changes from 3 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
aef5b14
test-file-added
shivaji-dev1 Aug 18, 2025
054f4be
update in hello.py
shivaji-dev1 Aug 18, 2025
c17f499
update 3
shivaji-dev1 Aug 18, 2025
9e94d47
update 4
shivaji-dev1 Aug 18, 2025
9663edb
update 5
shivaji-dev1 Aug 18, 2025
3b12348
update 6
shivaji-dev1 Aug 18, 2025
6fe7b7f
update 7
shivaji-dev1 Aug 18, 2025
7406c6c
update 7
shivaji-dev1 Aug 18, 2025
52da1d2
update 9
shivaji-dev1 Aug 18, 2025
efa41fb
update 10
shivaji-dev1 Aug 18, 2025
b783758
update 11
shivaji-dev1 Aug 18, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions app/hello.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
print("Hello world")
print("Hello world 2")
print("Hello world 3")
Copy link

Choose a reason for hiding this comment

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

Missing Main Guard

Code executes immediately when imported as a module, potentially causing unintended side effects in larger applications. This could lead to accidental execution of code in production environments or during testing.

Suggested change
print("Hello world")
print("Hello world 2")
print("Hello world 3")
def main():
print("Hello world")
print("Hello world 2")
print("Hello world 3")
if __name__ == "__main__":
main()
Standards
  • CWE-710

Copy link

Choose a reason for hiding this comment

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

Code runs on import without main guard

Code executes when imported as a module, which can cause unintended side effects in larger applications. Without a main guard, importing this file will trigger print statements, potentially disrupting logging or output in production environments.

Suggested change
print("Hello world")
print("Hello world 2")
print("Hello world 3")
def main():
print("Hello world")
print("Hello world 2")
print("Hello world 3")
if __name__ == "__main__":
main()
Standards
  • ISO-IEC-25010-Reliability-Maturity
  • SRE-Modularity
  • DbC-Interface-Contracts