Skip to content

fix: disable test parallelization 馃槩 - #401

Merged
jfmengels merged 4 commits into
jfmengels:mainfrom
lishaduck:maxworkers
Dec 19, 2025
Merged

fix: disable test parallelization 馃槩#401
jfmengels merged 4 commits into
jfmengels:mainfrom
lishaduck:maxworkers

Conversation

@lishaduck

Copy link
Copy Markdown
Contributor

Maybe someday we'll solve #324...

See #313 (comment)
Fixes #313.

@lishaduck

Copy link
Copy Markdown
Contributor Author

Note to self: if this ends up working well, revert #365 (esp. since this slows down tests slightly)

Looks like jfmengels#385 finally simplified how strings get injected, which broke the regex.
But also it's nicer like this so not complaining much. This *is* pretty hacky.
Comment thread package.json
"eslint-check": "eslint . --report-unused-disable-directives --max-warnings=0",
"eslint-fix": "npm run eslint-check -- --fix",
"jest": "jest",
"jest": "jest --detectLeaks",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Somehow I forgot #381 hadn't landed 馃槄
This isn't bad to have but it isn't doing much until #381 lands.

@jfmengels
jfmengels merged commit 3144dea into jfmengels:main Dec 19, 2025
3 checks passed
@jfmengels

Copy link
Copy Markdown
Owner

Maybe someday we'll solve #324...

馃

Thanks for the PR!

@lishaduck
lishaduck deleted the maxworkers branch December 19, 2025 15:18
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.

Flaky test

2 participants