Skip to content

fix: remove unnecessary global statement in Rock-Paper-Scissor.py#1639

Closed
ashrion wants to merge 1 commit into
steam-bell-92:mainfrom
ashrion:fix/rps-remove-unnecessary-globals
Closed

fix: remove unnecessary global statement in Rock-Paper-Scissor.py#1639
ashrion wants to merge 1 commit into
steam-bell-92:mainfrom
ashrion:fix/rps-remove-unnecessary-globals

Conversation

@ashrion

@ashrion ashrion commented Jul 11, 2026

Copy link
Copy Markdown

Summary

Removed unnecessary global statement from Rock-Paper-Scissor.py. The function declared 34 variables as global but none are used outside main().

Changes

  • Removed 34-variable global declaration from main()

How to Test

  1. Run python games/Rock-Paper-Scissor/Rock-Paper-Scissor.py
  2. Play a few rounds to verify game works correctly
  3. Confirm no NameError or behavior changes

Related Issues

Fixes #1638

GSSoC 2026

Removed 34-variable global declaration from main() since all variables
are local and never accessed outside the function.

Fixes steam-bell-92#1638
@vercel

vercel Bot commented Jul 11, 2026

Copy link
Copy Markdown

@ashrion is attempting to deploy a commit to the Anuj's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

fix: remove unnecessary global statement in Rock-Paper-Scissor.py

2 participants