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

feat(config): import mud config with tsx #3290

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

feat(config): import mud config with tsx #3290

wants to merge 5 commits into from

Conversation

holic
Copy link
Member

@holic holic commented Oct 14, 2024

hoping this solves the race condition in failing tests in main when running tests in parallel, where each test imports a MUD config and thus tries to compile the file to the same path

I tried ts-import and esbuild-register before landing on a working solution with tsx

Copy link

changeset-bot bot commented Oct 14, 2024

🦋 Changeset detected

Latest commit: af959be

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 24 packages
Name Type
@latticexyz/config Patch
@latticexyz/cli Patch
@latticexyz/explorer Patch
@latticexyz/protocol-parser Patch
@latticexyz/stash Patch
@latticexyz/store-sync Patch
@latticexyz/store Patch
@latticexyz/world-modules Patch
@latticexyz/world Patch
@latticexyz/store-indexer Patch
@latticexyz/dev-tools Patch
@latticexyz/react Patch
@latticexyz/world-module-metadata Patch
@latticexyz/abi-ts Patch
@latticexyz/block-logs-stream Patch
@latticexyz/common Patch
create-mud Patch
@latticexyz/faucet Patch
@latticexyz/gas-report Patch
@latticexyz/recs Patch
@latticexyz/schema-type Patch
solhint-config-mud Patch
solhint-plugin-mud Patch
@latticexyz/utils Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@holic
Copy link
Member Author

holic commented Oct 18, 2024

blocked-ish on the same world TS issues we keep running into

might be time to transition abi-ts to generate a .ts file instead of .d.ts

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.

1 participant