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

Adding in setup scripts for Windows powershell users #867

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

Conversation

jwoertink
Copy link
Member

@jwoertink jwoertink commented Sep 8, 2024

Fixes #872

This adds in the next step to Windows compatibility. When you generate a Lucky app on Windows, it will now generate PowerShell scripts for you to run making the flow feel similar to mac/linux.

Full disclaimer... I used AI to translate bash to powershell then followed it up with lots of stackoverflow posts 😂 I'll try to run this on an actual Windows machine to verify, but I'm just getting the PR up so we can poke around with it.

@jwoertink
Copy link
Member Author

'D:\\a\\lucky_cli\\lucky_cli\\spec/../fixtures\\src_template\\expected\\script/setup.ps1': The system cannot find the file specified.

My guess is the fixtures aren't setup to handle switching formats since they are statically generated. The weird thing though is why do the slashes go both directions? It says script/setup.ps1, but I'm using Path.new which should be setup for Windows properly...

@jwoertink jwoertink added the hacktoberfest-accepted PRs accepted for Hacktoberfest label Oct 11, 2024
@jwoertink
Copy link
Member Author

Well... I now have new errors which is good, but I still don't quite understand them.

@jwoertink
Copy link
Member Author

I had a thought on this... what if instead of using bash/powershell, it was just a Crystal bootstrap? Would be nice if it could just be Ruby or something to make it a lot faster, but that means a new language requirement. We at least know Crystal will exist when this runs, and things like yarn can just run through Process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted PRs accepted for Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Port bash scripts to PowerShell for Windows
1 participant