Skip to content

Add interactive dice_roller.py implementation (Closes #3202) - #3216

Open
Bhunesh386 wants to merge 2 commits into
geekcomputers:masterfrom
Bhunesh386:feature/dice-roller
Open

Add interactive dice_roller.py implementation (Closes #3202)#3216
Bhunesh386 wants to merge 2 commits into
geekcomputers:masterfrom
Bhunesh386:feature/dice-roller

Conversation

@Bhunesh386

@Bhunesh386 Bhunesh386 commented Aug 16, 2026

Copy link
Copy Markdown

Description

Please include a summary of the changes and related context. Explain the problem you're solving or feature you're adding.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code cleanup/refactoring

Related Issues

Closes #(issue number)

Testing

Please describe the tests you've run to verify your changes:

  • Manual testing
  • Added automated tests
  • Existing tests pass

Checklist

  • My code follows the project's code style
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the README.md if needed
  • My changes generate no new warnings
  • I have tested my code locally
  • New and existing tests pass with my changes

Screenshots (if applicable)

Add screenshots or demos of your changes if relevant.

Copilot AI lite review requested due to automatic review settings August 16, 2026 20:07

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an interactive command-line dice roller script, generating box-drawing dice faces side-by-side and summarizing the roll results.

Changes:

  • Refactors dice-face rendering into a helper (get_die_art) and wraps execution in a main() entry point.
  • Adds input validation for number of dice (1–10) and prints results + total.
  • Updates the ASCII/box-drawing dice art rendering to print multiple dice horizontally.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread dice_roller.py Outdated
Comment thread dice_roller.py
Comment on lines +9 to +11
faces = {
1: ["┌───────┐", "│ │", "│ ● │", "│ │", "└───────┘"],
2: ["┌───────┐", "│ ● │", "│ │", "│ ● │", "└───────┘"],
Comment thread dice_roller.py
Comment on lines +20 to +23
def main():
print("========================================")
print(" Interactive Dice Roller ")
print("========================================")
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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.

2 participants