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

Fixed bug in Ultimate Tic Tac Toe termination conditions #1307

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

armatthews
Copy link

Fixed bug in Ultimate Tic Tac Toe implementation that caused games to end when any small board was terminal instead of when all small boards are terminal

… end when any small board was terminal instead of when *all* small boards are terminal
@lanctot
Copy link
Collaborator

lanctot commented Feb 12, 2025

Thanks!

@lanctot
Copy link
Collaborator

lanctot commented Feb 12, 2025

Regarding the failing tests: seems like a new issue in Dark Hex is causing a compilation problem that wasn't happening before.

I am on vacation at the moment but will take a look when I get back next week. Once we fix it you can just pull the changes from master and retrigger the tests.

@lanctot
Copy link
Collaborator

lanctot commented Feb 18, 2025

Regarding the failing tests: seems like a new issue in Dark Hex is causing a compilation problem that wasn't happening before.

I am on vacation at the moment but will take a look when I get back next week. Once we fix it you can just pull the changes from master and retrigger the tests.

Ok, this fix has been merged into master. Can you pull changes from master and push the merge commit to trigger the tests again?

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