Skip to content

Fix a windows import bug#88

Merged
ef4 merged 1 commit intomainfrom
test-loading
Mar 28, 2025
Merged

Fix a windows import bug#88
ef4 merged 1 commit intomainfrom
test-loading

Conversation

@ef4
Copy link
Copy Markdown
Contributor

@ef4 ef4 commented Mar 28, 2025

import-meta-resolve sometimes returns paths instead of file: URLs.

Windows sometimes throws if you try to import() an absolute path that is not a file URL.

I couldn't reproduce this in the test suite, I suspect Jest's module shenanigans.

`import-meta-resolve` sometimes returns paths instead of file: URLs.

Windows sometimes throws if you try to `import()` an absolute path that is not a file URL.

I couldn't reproduce this in the test suite, I suspect Jest's module shenanigans.
@ef4 ef4 added the bug Something isn't working label Mar 28, 2025
@ef4 ef4 changed the title Test module loading Fix a windows import bug Mar 28, 2025
@ef4 ef4 merged commit 2c5e141 into main Mar 28, 2025
10 checks passed
@github-actions github-actions Bot mentioned this pull request Mar 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant