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

fix errors for contributors on Windows #747

Merged
merged 2 commits into from
Nov 12, 2023

Conversation

wirekang
Copy link
Contributor

scripts/module-fixup.js

Since node 'path' module uses platform specific path separator, git-ignored helpers/ files could be generated with escape character \:
export * from '..\dist\cjs\helpers\postgres.js'

Updated path -> path.posix to use / as path separator always.

New dev-dependency

cross-env: for setting environment variables regardless of platform while testing.


I found this errors recently by moving primary OS from Ubuntu to Windows.
I can't completly sure this is Windows-related because it's hard to believe that out of 71 contributors, none of them use Windows.

@vercel
Copy link

vercel bot commented Oct 25, 2023

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 Nov 7, 2023 11:06am

@igalklebanov
Copy link
Member

I can't completly sure this is Windows-related because it's hard to believe that out of 71 contributors, none of them use Windows.

I use WSL2, so I never had this issue.

@igalklebanov igalklebanov added ready for review enhancement New feature or request tests and removed enhancement New feature or request labels Nov 3, 2023
@igalklebanov igalklebanov merged commit 2365556 into kysely-org:master Nov 12, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants