Skip to content
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.

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

User input validation & docstring update #625

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

sdthinlay
Copy link
Contributor

@sdthinlay sdthinlay commented Nov 1, 2023

Type of change

  • Added new project
  • Bug fix
  • New features
  • Enhanced documentation

Changes proposed in this pull request

  • Project name: Loan Calculator

  • Short description: Initially the game failed to evaluate invalid user input so with the updated code the user input is validated, test case is also added with docstring update.

Checklist

  • The code is Python 3 compatible
  • I have added relevant comments in my file to make code understandable
  • I have added my project in a folder which includes all the necessary files

Copy link
Contributor

github-actions bot commented Nov 1, 2023

Greetings! This is an automated message from GitHub Actions. 🤖
Your pull request has been received and is awaiting for a review by the repository owner or a maintainer. This may take some time, so please be patient.
While you wait, you can continue to work on other issues or pull requests, or explore the project further. Or you can simply relax and enjoy your day.
Thank you for your contribution to this project! You are awesome! ⭐

@Mrinank-Bhowmick
Copy link
Owner

Please maintain project directory structure

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.

None yet

2 participants