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

Zillion: change test detection for running tests with multiprocessing #2243

Conversation

beauxq
Copy link
Collaborator

@beauxq beauxq commented Sep 30, 2023

What is this fixing or adding?

change test detection for running tests with multiprocessing
As requested by @black-sliver

In addition to the changes to the detection, I also moved the detection function into the zilliandomizer package so it can use the same detection in both places.

How was this tested?

unit tests in both places and generated and played through a 1-player game in both places

Copy link
Member

@black-sliver black-sliver left a comment

Choose a reason for hiding this comment

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

Thanks!

@black-sliver black-sliver merged commit d5d630d into ArchipelagoMW:main Oct 1, 2023
12 checks passed
@ThePhar ThePhar added the is: enhancement Issues requesting new features or pull requests implementing new features. label Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: enhancement Issues requesting new features or pull requests implementing new features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants