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

Improve ./start to work on Windows #2421

Merged
merged 2 commits into from
Dec 18, 2024
Merged

Conversation

Eric-Arellano
Copy link
Collaborator

Closes #1607. There are two issues:

  1. The shebang will be ignored, so you need to run python ./start instead of ./start
  2. Volume mounting with Docker on Windows uses Unix-style path, per https://medium.com/@kale.miller96/how-to-mount-your-current-working-directory-to-your-docker-container-in-windows-74e47fa104d7

@Eric-Arellano
Copy link
Collaborator Author

TODO: confirm with @beckykd this fixes things.

@Eric-Arellano
Copy link
Collaborator Author

Eric-Arellano commented Dec 3, 2024

@frankharkins @arnaucasau before landing this, please see if the status quo works with Git Bash for Becky. If it does, please confirm this change does not break Git Bash and it also fixes PowerShell.

Imo supporting PowerShell is not essential, so long as Git Bash works on Windows. No worries if this PR doesn't fix PowerShell, so long as Git Bash works.

README.md Outdated Show resolved Hide resolved
@Eric-Arellano Eric-Arellano changed the title [wip] Improve ./start to work on Windows Improve ./start to work on Windows Dec 17, 2024
@Eric-Arellano Eric-Arellano marked this pull request as ready for review December 17, 2024 19:10
@Eric-Arellano
Copy link
Collaborator Author

We got it working on both Powershell and Git Bash 🚀

@Eric-Arellano
Copy link
Collaborator Author

@arnaucasau @frankharkins bump on reviewing this please

Copy link
Collaborator

@arnaucasau arnaucasau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! I tested the script on a Windows machine using PowerShell and Git Bash and worked well! Thanks 🚀

@Eric-Arellano Eric-Arellano added this pull request to the merge queue Dec 18, 2024
Merged via the queue into main with commit fbd0875 Dec 18, 2024
2 checks passed
@Eric-Arellano Eric-Arellano deleted the EA/start-windows-support branch December 18, 2024 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Improve Windows support for local previews
3 participants