Skip to content

Fix to Issue #163#169

Open
NathanKim16 wants to merge 1 commit intoufosc:mainfrom
NathanKim16:main
Open

Fix to Issue #163#169
NathanKim16 wants to merge 1 commit intoufosc:mainfrom
NathanKim16:main

Conversation

@NathanKim16
Copy link
Copy Markdown

@NathanKim16 NathanKim16 commented Feb 18, 2026

Type of Change

  • [ X] Feature
  • Bug Fix
  • Update
  • Refactor
  • Other (please describe)

Related Issue

Fixes issue #163

Changes

Changed player.py to include stat counters, and changed uno.py by inserting stat incrementors when necessary as well as displaying the stats at the end of the game.

Please include:

  • files edited or added
  • description of changes made in each file

These were necessary to implement the requirement from the issue which was to add a stats page at the end of the uno game

Testing

These changes were tested by playing the game

If this is a bug fix, please provide steps to reproduce the bug

Checklist

  • No merge conflicts
  • [ X] I self-reviewed my code
  • I added comments to make my code readable
  • I added tests to cover my changes
  • I have checked to see there are no open pull requests for the same issue

@NeedToLookAtMyPictures
Copy link
Copy Markdown
Collaborator

NeedToLookAtMyPictures commented Feb 23, 2026

I played through a game of UNO. After winning, there was no prompt or winner menu presented.
Can you add:

  • a post-game stat window that the user can toggle to after victory
  • a hotkey that allows the user to flip between player stat pages (since they should all be tracked).

If the second requirement is too cumbersome to implement, we can make it a separate issue.

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