Skip to content

Update Recommendation For Testing#2520

Merged
fenhl merged 1 commit into
OoTRandomizer:Devfrom
jdunn596:notes
Feb 19, 2026
Merged

Update Recommendation For Testing#2520
fenhl merged 1 commit into
OoTRandomizer:Devfrom
jdunn596:notes

Conversation

@jdunn596

@jdunn596 jdunn596 commented Jan 27, 2026

Copy link
Copy Markdown

The -n flag isn't compatible with pytest alone and requires the extension pytest-xdist to function. As a result, using the provided config while following the current instructions will cause pytest to fail. Installing pytest-xdist will also install pytest as a dependency, resolving this issue.

Testing

Ran tests with provided config post pytest-xdist install

@fenhl fenhl added Status: Needs Review Someone should be looking at it Component: Documentation Affects user-facing help messages or public API docs Type: Maintenance Code style, infrastructure, updating dependencies labels Jan 29, 2026

@cjohnson57 cjohnson57 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Confirmed your findings

@cjohnson57 cjohnson57 removed the Status: Needs Review Someone should be looking at it label Feb 18, 2026
@fenhl fenhl added this to the next milestone Feb 19, 2026
@fenhl fenhl merged commit 0b2d730 into OoTRandomizer:Dev Feb 19, 2026
6 checks passed
@jdunn596 jdunn596 deleted the notes branch April 5, 2026 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: Documentation Affects user-facing help messages or public API docs Type: Maintenance Code style, infrastructure, updating dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants