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

Modularization code and add unit test to Quiz Game #764

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

Conversation

Sae-Inoue
Copy link

@Sae-Inoue Sae-Inoue commented Jun 9, 2024

Type of change

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

Changes proposed in this pull request

  • Project name : Quiz Game

  • Short description : Modularization code. I created 2 modules gamesystem.py and gameplay.py so that we can maintenance efficiency. In addition, I created the unit test to make test easy. There were no docstrings, so added docstrings as well.

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 Jun 9, 2024

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! ⭐

@Sae-Inoue Sae-Inoue changed the title Issue 734 Modularization code and add unit test to Quiz Game Jun 9, 2024
@Sae-Inoue
Copy link
Author

I changed the project directory name to import module. Is that okay?

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