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

Possible Fix to Issue #768 ERR_UNSUPPORTED_ESM_URL_SCHEME on Windows and other posix #991

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

czuma
Copy link

@czuma czuma commented May 17, 2024

i was working on a solution for this bug because work on windows and deploy on linux

i tested the solution on windows and ubuntu via (vm), works fine but only if package.json type is set to "module" don't know why the cjs commonJS build don't give me the right result.
i run the test on ubuntu and don't give me errors on the migration files so guess is "ok"

maybe i commit an error? check it and let me know, maybe we can fix this.

ERR_UNSUPPORTED_ESM_URL_SCHEME on Windows
Copy link

vercel bot commented May 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
kysely ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 17, 2024 4:45am

@koskimas
Copy link
Member

Any solution has to work with both ESM and CJS as well as on all js runtimes. Not just node.

@igalklebanov
Copy link
Member

igalklebanov commented May 18, 2024

Try passing https://github.com/unjs/pathe to constructor.

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.

None yet

3 participants