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

add rps cmd && add game category #48

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

Conversation

ademyro
Copy link
Contributor

@ademyro ademyro commented Jan 7, 2023

What kind of change does this PR introduce?
This Pull-Request introduces a new command - RPS, or Rock Paper Scissors.
To try it out, it's simple: v!rps
Then follow the instructions.

It also introduces a new game category (prefix = None) in which tic_tac_toe.py and rps.py are in.

Describe the changes proposed in the pull request
Just a simple command with an additional category.

What is the current behavior?
I tested it multiple times (e.g. reacting to a different message, wrong reaction etc) and I didn't encounter any issue.
But it's still possible that some feature does not work as intended.

Does this PR introduce a breaking change?
If you're going to add a game command, please add it in the game

@logicguy1 logicguy1 added the Pending Changes This pull request is pending changes made by the autor in response to a review label Jan 8, 2023
@ademyro
Copy link
Contributor Author

ademyro commented Feb 5, 2023

What should I change to this pull-request? (I'm sorry if it's something obvious)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pending Changes This pull request is pending changes made by the autor in response to a review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants