Skip to content

added input validation#103

Open
krishnashrimali88 wants to merge 2 commits into
PrajaktaSathe:mainfrom
krishnashrimali88:main
Open

added input validation#103
krishnashrimali88 wants to merge 2 commits into
PrajaktaSathe:mainfrom
krishnashrimali88:main

Conversation

@krishnashrimali88

Copy link
Copy Markdown

What does this PR do?

This PR adds input validation to the Rock Paper Scissors script. Previously.
if a user entered anything other than Rock, Paper, or Scissors, the script would silently loop back to the prompt with no feedback.

Related PRs and Issues is any

Have you read the Contributing Guidelines?

  • [ x] yes
  • no

Is the code your own?

  • [x ] yes
  • no

Is your code in proper folder and proper subfolder?

  • [x ] yes
  • no

Have you made or updated corresponding README.md file in the folder?

  • [x ] yes
  • no

Is the code beautified?

  • [x ] yes
  • no

Is the code clear to understand with proper variable naming and comments?

  • [ x] yes
  • no

Is the problem/project already present and you have implemented it using a new technique?

  • yes, i have used a new technique and added corresponding documentation
  • [ x] no, it is new

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