Skip to content

Dice Roll #121

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

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

Dice Roll #121

wants to merge 3 commits into from

Conversation

Thromax
Copy link
Contributor

@Thromax Thromax commented Sep 14, 2021

Closes #85

Summary

Added !roll command.
Users can now roll a set of numbers by using any of the following syntax:
!roll <number> : Will return same number
!roll d<faces> : Rolls a single -faced dice
!roll <dice>d<faces> : Rolls -faced dice

Checklist

  • I have read and agree to follow the contributing guidelines and code of conduct in CONTRIBUTING.md.
  • An issue was made and assigned to me by a maintainer before opening this pull request.
  • I have formatted the code as detailed in CONTRIBUTING.md.
  • I have tested all the code in my pull request, including by running BrainBot as a whole.

@Thromax Thromax requested a review from a team as a code owner September 14, 2021 22:40
@Thromax Thromax requested review from bsoyka and LukeG294 September 14, 2021 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Dice Roll
1 participant