search

jacobparis-insiders / sly

Monorepo for Sly CLI
https://sly-cli.fly.dev/
153 stars 11 forks source link

CLI doesn't work on Windows #13

Closed wKovacs64 closed 8 months ago

wKovacs64 commented 8 months ago

The bin script for the CLI is a bash shell script, which doesn't work on Windows (without users doing things you can't really expect them to do). Can we convert it to a Node script so Sly CLI works cross-platform?

I played around with some solutions and have it working locally by simply pointing bin at ./dist/index.js directly as I didn't see the need for a TS loader at all, but I also have a version using tsx instead of your custom ts-loader.js in case there really is a need to process TS files.

Thoughts? Happy to open a PR if you want.

Thanks.

jacobparis commented 8 months ago

Have you tried on the latest version (1.5.0)? This issue might be related

https://github.com/jacobparis-insiders/sly/issues/7

The use-case for the TS Loader is allowing users to specify transformer files in Typescript

wKovacs64 commented 8 months ago

Have you tried on the latest version (1.5.0)? This issue might be related

Yep, I'm using 1.5.0. I don't think it has anything to do with transforms, it's a matter of being able to run the shell script at all.

The use-case for the TS Loader is allowing users to specify transformer files in Typescript

Ah, gotcha. 👍